Skip to content

590 bug identityrevalidatingprovider#591

Merged
PTKu merged 3 commits into
devfrom
590-bug-identityrevalidatingprovider
Feb 11, 2025
Merged

590 bug identityrevalidatingprovider#591
PTKu merged 3 commits into
devfrom
590-bug-identityrevalidatingprovider

Conversation

@kuh0005
Copy link
Copy Markdown
Collaborator

@kuh0005 kuh0005 commented Feb 11, 2025

This pull request includes several changes to the security integration and authentication state provider functionalities. The most important changes include adding new JSON records for admin groups and users, modifying the IdentityRevalidatingAuthenticationStateProvider to use a normalizer, and updating the MongoDB repository setup.

Security Integration Updates:

  • Added a new JSON record for the AdminGroup with roles and timestamps in src/Security/integrations/JSONREPOS/Groups/AdminGroup.
  • Added a new JSON record for the ADMIN user with detailed attributes and security settings in src/Security/integrations/JSONREPOS/Users/ADMIN.

Authentication State Provider Enhancements:

  • Removed the static Instances counter and added UpperInvariantLookupNormalizer in IdentityRevalidatingAuthenticationStateProvider.cs.
  • Updated the ValidateAuthenticationStateAsync method to use the normalizer for user name lookup in IdentityRevalidatingAuthenticationStateProvider.cs.
  • Corrected a typo in the exception message within the ValidateAuthenticationStateAsync method in IdentityRevalidatingAuthenticationStateProvider.cs.

MongoDB Repository Setup:

  • Refactored the MongoDB repository setup to use AXOpen.Data.MongoDb.Repository.Factory for creating user and group repositories in src/security/integrations/axopen_blazor_auth_app/Program.cs.

closes #590

@kuh0005 kuh0005 linked an issue Feb 11, 2025 that may be closed by this pull request
@kuh0005 kuh0005 requested a review from PTKu February 11, 2025 08:25
@PTKu PTKu merged commit 00b6212 into dev Feb 11, 2025
@PTKu PTKu deleted the 590-bug-identityrevalidatingprovider branch February 11, 2025 09:22
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.

[BUG] IdentityRevalidatingProvider

2 participants