Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Mirth Connect Login with LDAP Plugin is Case-sensitive for Username #5396

Closed
jdonextgen opened this issue Sep 1, 2022 · 3 comments
Closed
Labels
bug Something isn't working Fix-Commited Issue fixed and will be available in milestone Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-81
Milestone

Comments

@jdonextgen
Copy link
Collaborator

jdonextgen commented Sep 1, 2022

Describe the bug
Users are unable to use their LDAP login unless they used all lowercase characters for their username. This bug is reproducible.

To Reproduce

  1. Setup LDAP
  2. Create credentials in LDAP where the username contains at least one capital letter.

Steps to reproduce the behavior:

  1. Setup Mirth Connect with LDAP
  2. Login using credentials created in LDAP
  3. See error

Expected behavior
Users should be able to to use their LDAP login with a case-insensitive username.

Actual behavior
Users are unable to use their LDAP login unless they used all lowercase characters for their username.

Screenshots
N/A

Environment (please complete the following information):

  • OS: [e.g. Linux (CentOS), Windows 10, MacOS]
  • Java Distribution/Version [e.g. OpenJDK 11, Java 8 (201)]
  • Connect Version [e.g. 3.8.0]: 3.10.1

Workaround(s)

  1. Login with LDAP credentials using all lowercase for the username
    OR
  2. Login as a user with privileges to modify users
  3. Modify username of LDAP user to match the casing in the LDAP username
  4. Logout and login using LDAP credentials with the specified casing

Additional context
N/A

@jdonextgen jdonextgen added bug Something isn't working Fix-Commited Issue fixed and will be available in milestone Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-81 labels Sep 1, 2022
@jonbartels
Copy link
Contributor

Holy crap! It finally got fixed!

@jonbartels
Copy link
Contributor

jonbartels commented Sep 2, 2022

@jdonextgen This fixes KI197032 and KI177637 right? If it does please update those KIs and please highlight this issue in the release notes. I don't know if I can truly convey what a big deal this bug was.

The underlying problem that my team observed externally was that LDAP did a search (case sensitive) before trying the bind (not case sensitive). Can you roughly describe the implemented solution? Do you still do the search or just jump straight to the bind like most other LDAP tools do?

@jdonextgen jdonextgen added this to the 4.2.0 milestone Sep 2, 2022
@jdonextgen
Copy link
Collaborator Author

@jonbartels Yeah, this bug fix should fix KI197032 and KI177637. From what I've been told, we still do the search but it's now case-insensitive.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Fix-Commited Issue fixed and will be available in milestone Internal-Issue-Created An issue has been created in NextGen's internal issue tracker RS-81
Projects
None yet
Development

No branches or pull requests

3 participants