Skip to content

Commit

Permalink
Ignore case when comparing claim types (#205)
Browse files Browse the repository at this point in the history
* ignore case when comparing claim types

* Update CHANGELOG.md

* Update CHANGELOG.md
  • Loading branch information
Yvand authored Apr 15, 2024
1 parent ce3ae05 commit 4e2896b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,12 @@

## Unreleased

* Fix error when creating the configuration, due to case-sensitive test in the claim types - https://github.com/Yvand/LDAPCP/issues/204
* Fix the error when loading the global configuration page, if the group claim type set in the LDAPCP configuration does not exist in the trust. https://github.com/Yvand/LDAPCP/issues/203

## LDAPCP Second Edition v17.0.20240226.2 - Published in February 26, 2024

* Initial release of LDAPCP Second Edition, a complete rewrite of current project
* Ignore case when comparing claim types, to avoid errors when creating the configuration - https://github.com/Yvand/LDAPCP/pull/205

## LDAPCP v16.0.20230824.1 enhancements & bug-fixes - Published in August 24, 2023

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ public static ClaimTypeConfigCollection ReturnDefaultClaimTypesConfig(string cla


//// Not adding those as additional attributes to avoid having too many LDAP attributes to search users in the LDAP filter
var nonIdentityClaimTypes = ClaimsProviderConstants.GetDefaultSettingsPerUserClaimType().Where(x => x.Key != spTrust.IdentityClaimTypeInformation.MappedClaimType);
var nonIdentityClaimTypes = ClaimsProviderConstants.GetDefaultSettingsPerUserClaimType().Where(x => !String.Equals(x.Key, spTrust.IdentityClaimTypeInformation.MappedClaimType, StringComparison.OrdinalIgnoreCase));
foreach (var nonIdentityClaimType in nonIdentityClaimTypes)
{
ctConfig = nonIdentityClaimType.Value;
Expand Down

0 comments on commit 4e2896b

Please sign in to comment.