Skip to content

fix(auth): make user identifier comparison case-insensitive#246

Merged
bosschaert merged 1 commit intomainfrom
emailcs
Feb 25, 2026
Merged

fix(auth): make user identifier comparison case-insensitive#246
bosschaert merged 1 commit intomainfrom
emailcs

Conversation

@bosschaert
Copy link
Contributor

Description

Make user identification (email, IMS org and group) for ACL permissions case insensitive.

So for example, if a user used a different case in the permissions sheet to specify an email compared to how identifiers are registered in IMS comparison is done in a case-insensitive way.

Added extra tests as well, also around the case-insensitive behaviour of the ACLTRACE, which was partly in place already.

Related Issue

Fixes: adobe/da-live#421

How Has This Been Tested?

New and existing unit tests.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@bosschaert bosschaert merged commit aac2136 into main Feb 25, 2026
5 checks passed
@bosschaert bosschaert deleted the emailcs branch February 25, 2026 15:34
adobe-bot pushed a commit that referenced this pull request Feb 25, 2026
## [1.1.6](v1.1.5...v1.1.6) (2026-02-25)

### Bug Fixes

* **auth:** make user identifier comparison case-insensitive ([#246](#246)) ([aac2136](aac2136))
@adobe-bot
Copy link
Collaborator

🎉 This PR is included in version 1.1.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[permissions] check for email addresses should be caseless

3 participants