Skip to content

[release/11.0] Harden race condition when submitting 2FA during password reset - #69062

Open
github-actions[bot] wants to merge 4 commits into
release/11.0from
backport/pr-68350-to-release/11.0
Open

[release/11.0] Harden race condition when submitting 2FA during password reset#69062
github-actions[bot] wants to merge 4 commits into
release/11.0from
backport/pr-68350-to-release/11.0

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Backport of #68350 to release/11.0

/cc @Youssef1313

Harden race condition when submitting 2FA during password reset

Hardens a race condition when password is reset in the time window between requesting 2FA and submitting it.

Description

There is a niche race condition when a password is reset after 2FA is requested and before 2FA is submitted. The 2FA submit is currently accepted even if the password was reset in between.

This PR hardens the implementation so that we validate the security stamp and reject the 2FA submission in this case.

Fixes #66824

Customer Impact

Hardening race condition. Not much impact is noticeable in all common cases. Only fixes a race in a niche scenario.

Regression?

  • Yes
  • No

Risk

  • Low

This adds additional validation for the 2FA security stamp. The code change is minimal and tested.

Verification

Automated.

Packaging changes reviewed?

N/A

Youssef1313 and others added 4 commits September 4, 2026 15:02
Co-authored-by: Korolev Dmitry <dmkorolev@microsoft.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@Youssef1313 Youssef1313 added the Servicing-consider Shiproom approval is required for the issue label Sep 4, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hi @github-actions[bot]. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@Youssef1313
Youssef1313 requested a review from artl93 September 4, 2026 15:11

@artl93 artl93 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reliability. Approved.

@artl93 artl93 added Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Servicing-approved Shiproom has approved the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants