Skip to content

Conversation

maliming
Copy link
Member

@maliming maliming commented Oct 7, 2025

No description provided.

@maliming maliming added this to the 10.1-preview milestone Oct 7, 2025
@maliming maliming requested a review from Copilot October 8, 2025 02:38
@maliming maliming marked this pull request as ready for review October 8, 2025 02:38
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a user password history entity to the ABP Identity module to support password reuse prevention functionality.

Key changes:

  • Introduces IdentityUserPasswordHistory entity to track user password history
  • Adds settings to configure password reuse prevention (enable/disable and count of previous passwords to check)
  • Updates Entity Framework and MongoDB contexts to support the new entity

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
IAbpIdentityMongoDbContext.cs Adds MongoDB collection property for password histories
AbpIdentityMongoDbContext.cs Implements MongoDB collection for password histories
IdentityEfCoreQueryableExtensions.cs Includes password histories in user detail queries
IdentityDbContextModelBuilderExtensions.cs Configures Entity Framework mapping for password history entity
IdentityUserPasswordHistory.cs Defines the password history entity with user ID and password as composite key
IdentityUser.cs Adds password history navigation property and helper method
AbpIdentitySettingDefinitionProvider.cs Adds settings for password reuse prevention configuration
AbpIdentityResultExtensions.cs Adds localization string for password reuse error message
IdentitySettingNames.cs Defines setting names for password reuse prevention
en.json Adds English localization for password reuse error message
IdentityUserPasswordHistoriesConsts.cs Defines constants for password history entity

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

maliming and others added 2 commits October 8, 2025 10:43
…y/IdentityUserPasswordHistory.cs

Co-authored-by: Copilot <[email protected]>
…y/IdentityUserPasswordHistory.cs

Co-authored-by: Copilot <[email protected]>
Copilot AI and others added 3 commits October 8, 2025 02:48
…ory-localization

Add PasswordInHistory localization text to all language files
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ maliming
❌ Copilot
You have signed the CLA already but the status is still pending? Let us recheck it.

@maliming maliming marked this pull request as draft October 8, 2025 03:41
@maliming maliming marked this pull request as ready for review October 8, 2025 08:07
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

❌ Patch coverage is 53.22581% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.09%. Comparing base (2750b3a) to head (7d63a83).
⚠️ Report is 64 commits behind head on dev.

Files with missing lines Patch % Lines
...n/Volo/Abp/Identity/IdentityUserPasswordHistory.cs 0.00% 21 Missing ⚠️
....Identity.Domain/Volo/Abp/Identity/IdentityUser.cs 22.22% 7 Missing ⚠️
.../Abp/Identity/MongoDB/AbpIdentityMongoDbContext.cs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##              dev   #23894   +/-   ##
=======================================
  Coverage   51.09%   51.09%           
=======================================
  Files        3269     3271    +2     
  Lines      106432   106500   +68     
  Branches     8173     8174    +1     
=======================================
+ Hits        54379    54420   +41     
- Misses      50379    50406   +27     
  Partials     1674     1674           

☔ View full report in Codecov by Sentry.
📢 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.

@maliming maliming requested a review from hikalkan October 9, 2025 06:22
@maliming maliming requested a review from EngincanV October 9, 2025 06:22
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.

4 participants