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] Username is case-sensitive when logging into MC Administrator #5569

Open
elavy-harris opened this issue Dec 7, 2022 · 6 comments
Open
Labels
bug Something isn't working

Comments

@elavy-harris
Copy link

Describe the bug
Login to Mirth Connect Administrator fails if the case of the username doesn't match.

To Reproduce

  1. Try to log in to MCA with different case on the letters.

Expected behavior
Login succeeds.

Actual behavior
Login fails

Environment (please complete the following information):

  • OS: Windows 10
  • Connect Version 4.1.1
@elavy-harris elavy-harris added the bug Something isn't working label Dec 7, 2022
@elavy-harris
Copy link
Author

Note: I am NOT using LDAP, so I don't think the fix in #5396 would fix my problem.

@pacmano1
Copy link
Collaborator

pacmano1 commented Dec 8, 2022

Is that a behavior change from the last release?

Curious really. Sounds like a new feature, lol.

Email addresses before the domain name by RFC are actually case sensitive by RFC 5321. Folks don't enforce it though.

@elavy-harris
Copy link
Author

I don't know if it's a change from prior versions. I happened to create a mixed-case username on 4.1.1 and then couldn't get signed on because I didn't know it mattered.

The official and actual behavior of email addresses isn't relevant, is it?

@pacmano1
Copy link
Collaborator

pacmano1 commented Dec 8, 2022

Not relevant, just a random fact.

@joshmc82
Copy link

I noticed this as well using 4.1.1 "ADMIN" vs "admin". Also tested on 3.10.1 and experienced the same thing.

@ChristopherSchultz
Copy link
Contributor

Hah. Multiply-confirmed on 4.1.1.

I'm using my own hand-rolled LDAP authentication plug-in and while the login worked (it's using LDAP), I was immediately presented with the "new user" dialog asking me to set up a password and give all my contact information to NextGen.

+1 to making usernames 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
Projects
None yet
Development

No branches or pull requests

4 participants