Skip to content

Ensure email is lowercase and check availability on user creation#5442

Open
ChinoKou wants to merge 1 commit intoNginxProxyManager:developfrom
ChinoKou:develop
Open

Ensure email is lowercase and check availability on user creation#5442
ChinoKou wants to merge 1 commit intoNginxProxyManager:developfrom
ChinoKou:develop

Conversation

@ChinoKou
Copy link
Copy Markdown

This pull request adds email normalization and uniqueness validation to the user creation logic in backend/internal/user.js. Now, emails are converted to lowercase and trimmed, and the system checks if the email is already in use before proceeding.

User validation improvements:

  • Emails are now normalized (lowercased and trimmed) before processing to ensure consistency.
  • Added a uniqueness check for emails during user creation, throwing a validation error if the email is already in use.

@ChinoKou
Copy link
Copy Markdown
Author

Fixes: #4983 #5323

@ChinoKou ChinoKou marked this pull request as ready for review March 30, 2026 15:05
@nginxproxymanagerci
Copy link
Copy Markdown

Docker Image for build 1 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-5442

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant