Skip to content

Conversation

@idotta
Copy link
Owner

@idotta idotta commented Jan 2, 2026

Note

Standardizes multi-tenant strategy configuration and aligns codebase with new keys/defaults.

  • Replace option key constants with strategy-name keys in StrategyOptions and rename defaults: DefaultHeaderNameDefaultHeader, DefaultClaimTypeDefaultClaim
  • Update tenant resolution wiring to use new keys in ServiceCollectionExtensions (WithHeaderStrategy, WithRouteStrategy, WithClaimStrategy)
  • Adjust ValidateBearerTokenTenantMiddleware, CurrentUserService, and IdmtUserClaimsPrincipalFactory to read claim/header names via new keys; make IsActive check case-insensitive
  • Extend sample appsettings*.json with StrategyOptions overrides for header and claim
  • Update tests to reflect new constants/keys and to read header name from IdmtOptions in IdmtApiFactory

Written by Cursor Bugbot for commit 9df8079. This will update automatically on new commits. Configure here.

idotta added 7 commits January 2, 2026 12:51
- Updated RegisterUser to retrieve tenant ID from the current user service, ensuring user registration is context-aware.
- Modified ValidateBearerTokenTenantMiddleware to validate tenant claims against the current request's tenant identifier.
- Enhanced CurrentUserService to include TenantIdentifier for better tenant context handling.
- Updated IdmtUserClaimsPrincipalFactory to ensure tenant claims are correctly generated and tested.
- Improved unit tests for CurrentUserService and IdmtUserClaimsPrincipalFactory to cover new tenant handling logic.
@idotta idotta merged commit e7e30d9 into main Jan 2, 2026
2 checks passed
@idotta idotta deleted the improve-strategies-logic branch January 2, 2026 22:45
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