search

OWASP / DevGuide

The OWASP Guide
Other
2.04k stars 382 forks source link

Updated introductory comment and fixed up references. Added most of C… #51

Closed kwwall closed 8 years ago

kwwall commented 8 years ago

Updated introductory comment and fixed up references. Added most of Chris Madden's new content on Key Management.

bradchesney79 commented 8 years ago

@kwwall @crashedmind

Well, done. It took me about an hour to read the whole thing and then a few parts again, but at the very least it is well written. Good wording choices and, as a guide, the material chosen is not too deep. I especially like how the images are handled versus my rudimentary knowledge from a little over a year ago. I had never even heard of TDE before.

kwwall commented 8 years ago

Thanks. There's still some stuff of Chris' (@crashedmind) that I want to add, but those parts are more difficult because they are either heavily interleaved with the existing material or need to be greatly summarized. I'd like to keep more, but the chapter is already pretty long.

-kevin Sent from my Droid; please excuse typos. On Mar 16, 2016 2:24 AM, "BradChesney79" notifications@github.com wrote:

@kwwall https://github.com/kwwall @crashedmind https://github.com/crashedmind

Well, done. It took me about an hour to read the whole thing and then a few parts again, but at the very least it is well written. Good wording choices and, as a guide, the material chosen is not too deep. I especially like how the images are handled versus my rudimentary knowledge from a little over a year ago. I had never even heard of TDE before.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/OWASP/DevGuide/pull/51#issuecomment-197170314