-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to Cred Stuffing Cheat Sheet #1224
Conversation
…address section and more.
… and other areas.
Removed the trailing space that caused a markdownlint failure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is amazing work. I left some minor suggestions but overall it's really great. Thank you so much for your contributions!
Co-authored-by: Shlomo Zalman Heigh <shlomozalmanheigh@gmail.com>
…swords referenced
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe I've addressed all open comments/requested changes, etc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you so much for this major improvement to the cheat sheet!
In this PR I've made a significant number of changes to the credential stuffing cheat sheet. The edits reflects a more up to date view on the wide array of commonly used defenses.
This PR covers issue #1208.