Skip to content
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

Forgot Password CS Rewrite #410

Merged
merged 57 commits into from Jun 22, 2020
Merged

Conversation

@ThunderSon
Copy link
Contributor

@ThunderSon ThunderSon commented Jun 6, 2020

This PR closes #159 and replaces #374

The forgot password CS focused heavily on the security questions.

In this rewrite, the CS focuses on:

  • multiple methods that could be used for the service
  • how to implement each method
  • common tasks to be conducted for all methods of forgot password
ThunderSon added 30 commits Dec 30, 2019
…/CheatSheetSeries into chore/forgot-password-rewrite
@ThunderSon ThunderSon self-assigned this Jun 6, 2020
@ThunderSon
Copy link
Contributor Author

@ThunderSon ThunderSon commented Jun 6, 2020

@rbsec Ready for review. Once content is accepted, I will work on the note-look from MkDocs, and some grammar improvements here and there.

Copy link
Contributor

@rbsec rbsec left a comment

I've made a few comments - would be good to chat about some of them on Slack at some point.

cheatsheets/Forgot_Password_Cheat_Sheet.md Outdated Show resolved Hide resolved
cheatsheets/Forgot_Password_Cheat_Sheet.md Outdated Show resolved Hide resolved
cheatsheets/Forgot_Password_Cheat_Sheet.md Outdated Show resolved Hide resolved
cheatsheets/Forgot_Password_Cheat_Sheet.md Outdated Show resolved Hide resolved
cheatsheets/Forgot_Password_Cheat_Sheet.md Outdated Show resolved Hide resolved
ThunderSon added 3 commits Jun 7, 2020
@ThunderSon
Copy link
Contributor Author

@ThunderSon ThunderSon commented Jun 16, 2020

Will finish this tomorrow. Having some thoughts about the MFA section.

ThunderSon added 5 commits Jun 20, 2020
@ThunderSon ThunderSon requested a review from rbsec Jun 20, 2020
ThunderSon and others added 6 commits Jun 21, 2020
rbsec and others added 3 commits Jun 21, 2020
Copy link
Contributor

@rbsec rbsec left a comment

Looks good, happy for this to be merged.

@rbsec rbsec dismissed their stale review via 06bfc88 Jun 21, 2020
@rbsec
rbsec approved these changes Jun 21, 2020
@ThunderSon ThunderSon merged commit 44328a5 into OWASP:master Jun 22, 2020
3 checks passed
3 checks passed
link-check
Details
lint
Details
Publishing Check
Details
@ThunderSon ThunderSon deleted the ThunderSon:forgot-password-rewrite branch Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants