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

Update Forgot_Password_Cheat_Sheet.md #1553

Merged
merged 3 commits into from
Dec 20, 2024
Merged

Update Forgot_Password_Cheat_Sheet.md #1553

merged 3 commits into from
Dec 20, 2024

Conversation

gl4nce
Copy link
Contributor

@gl4nce gl4nce commented Dec 3, 2024

  • removed section about backup codes from CS because it's out of scope
  • added hint for MFA recovery methods with link to corresponding CS

This PR fixes issue #1538.

Copy link
Member

@jmanico jmanico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also mention that these codes should be random?

Copy link
Collaborator

@kwwall kwwall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm fine with the general direction you are taking this, but can we please link to the specific section on "MFA Recovery" instead of just the general MFA CS?

@kwwall
Copy link
Collaborator

kwwall commented Dec 6, 2024

Should we also mention that these codes should be random?

Yes, but in the MFA Reset section rather than here.

@gl4nce gl4nce requested a review from kwwall December 6, 2024 12:37
@mackowski
Copy link
Collaborator

@kwwall @jmanico I think we should merge that one

Copy link
Collaborator

@kwwall kwwall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mackowski mackowski merged commit 4f303d6 into OWASP:master Dec 20, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants