nginx-proxy-manager icon indicating copy to clipboard operation
nginx-proxy-manager copied to clipboard

Login email address is case-sensitive

Open Ductapemaster opened this issue 4 years ago • 2 comments

Checklist

  • Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
    • Yes
  • Are you sure you're not using someone else's docker image?
    • Yes
  • Have you searched for similar issues (both open and closed)?
    • Yes

Describe the bug The email address field for used for login is case sensitive.

Nginx Proxy Manager Version v2.9.15

To Reproduce Steps to reproduce the behavior:

  1. Create an account with a capital letter in the user's email address, e.g. [email protected]
  2. Try to log in with the email address all in lower case e.g. [email protected]
  3. Login will be denied, No relevant user found

Expected behavior While email addresses are technically case sensitive (RFC 5321), providers do not usually enforce this. I believe the login handler should convert the input to lower case before comparing against the value stored in the database to

Operating System Raspberry Pi / Raspibian Buster Docker 20.10.5+dfsg1, build 55c4c88

Ductapemaster avatar Jan 25 '22 04:01 Ductapemaster

It really depends though.. Most modern email software actually DO implement case-sensitivity in a way. As they're RFC-compliant, they must. The way they get around it is: Make your own email addresses case-insensitive and lower-cased. Incoming mail will then get turned from [email protected] => [email protected]. Treat outgoing mail as case-sensitive (send a mail with [email protected] and it'll be sent as just that).

operinko avatar Jan 25 '22 08:01 operinko

I want to +1 this issue

Because this relates to the email login and not actually sending or receiving emails. Would this be able to be a configuration option to let the username check turn case-insensitive, even if it is changed after setup the stored emails should retain their case-sensitivity and have no effect on any other functionality or existing functionality if the default is set to how it acts now..

KlutzyBubbles avatar Jun 21 '22 13:06 KlutzyBubbles

Issue is now considered stale. If you want to keep it open, please comment :+1:

github-actions[bot] avatar Feb 26 '24 01:02 github-actions[bot]

Man so many issues closed from stale bots... I still want this (if it has already been implemented in a newer version I would also like to know about it)

KlutzyBubbles avatar Feb 28 '24 21:02 KlutzyBubbles