Skip to content

Fix password grant two-factor authentication flow - #26063

Open
maliming wants to merge 1 commit into
rel-10.7from
maliming/reset-2fa-failed-count-rel-10.7
Open

Fix password grant two-factor authentication flow#26063
maliming wants to merge 1 commit into
rel-10.7from
maliming/reset-2fa-failed-count-rel-10.7

Conversation

@maliming

Copy link
Copy Markdown
Member

Resolve #26052
Resolve #26057

Reset failed access counts after two-factor authentication and require two-factor authentication after password changes.

Copilot AI lite review requested due to automatic review settings August 25, 2026 04:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.11765% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.70%. Comparing base (bb4f483) to head (e567df5).
⚠️ Report is 9 commits behind head on rel-10.7.

Files with missing lines Patch % Lines
...spNetIdentity/AbpResourceOwnerPasswordValidator.cs 89.58% 3 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           rel-10.7   #26063      +/-   ##
============================================
+ Coverage     49.19%   49.70%   +0.50%     
============================================
  Files          3806     3813       +7     
  Lines        132496   132773     +277     
  Branches      10045    10073      +28     
============================================
+ Hits          65188    65995     +807     
+ Misses        65362    64783     -579     
- Partials       1946     1995      +49     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants