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

Fix 1151-2: Added section & info surrounding re-authN for sensitive functionality / identity changes. #1179

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

alp1n3-eth
Copy link
Contributor

@alp1n3-eth alp1n3-eth commented Jan 6, 2025

This PR covers issue #1151.

  • This PR handles the issue and requires no additional PRs.
  • You have validated the need for this change.

What did this PR accomplish?

Apologies for the delay on this. Hope you all had a nice holiday break!

Added the "Authenticated Identity and Configuration Changes" section to ensure any sensitive account changes require re-authentication.
Included name + username.
Copy link

github-actions bot commented Jan 6, 2025

The following issues were identified:
document/4-Web_Application_Security_Testing/04-Authentication_Testing/09-Testing_for_Weak_Password_Change_or_Reset_Functionalities.md:168:596 MD009/no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

This comment was marked as resolved.

@kingthorin
Copy link
Collaborator

The link check issue is a False Positive, it can be ignored.

…Testing/09-Testing_for_Weak_Password_Change_or_Reset_Functionalities.md

This comment was marked as resolved.

@kingthorin kingthorin merged commit 580d02b into OWASP:master Jan 10, 2025
2 of 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.

2 participants