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

fix: reject already existing usernames #443 #464

Merged
merged 15 commits into from
May 19, 2024
Merged

Conversation

ajnart
Copy link
Contributor

@ajnart ajnart commented May 10, 2024


Homarr

Motivation

Closes #443

Changes

  • Add a check when creating a user to see if a user with that name already exists. If that's the case then an error is thrown. ( 781247d )
  • Force lowercase usernames ( 400fa3d )

@ajnart ajnart requested a review from a team as a code owner May 10, 2024 11:42
@ajnart ajnart requested a review from Meierschlumpf May 10, 2024 11:42
@Meierschlumpf Meierschlumpf changed the title [443] Bugfix: Reject already existing users fix: reject already existing usernames #443 May 10, 2024
apps/nextjs/src/components/utils.tsx Outdated Show resolved Hide resolved
apps/nextjs/src/components/utils.tsx Outdated Show resolved Hide resolved
packages/api/src/router/user.ts Outdated Show resolved Hide resolved
packages/api/src/router/user.ts Outdated Show resolved Hide resolved
packages/api/src/router/user.ts Outdated Show resolved Hide resolved
packages/api/src/router/user.ts Outdated Show resolved Hide resolved
packages/api/src/router/user.ts Outdated Show resolved Hide resolved
manuel-rw
manuel-rw previously approved these changes May 11, 2024
packages/api/src/router/user.ts Outdated Show resolved Hide resolved
@Meierschlumpf Meierschlumpf added the bug Something isn't working label May 18, 2024
@ajnart
Copy link
Contributor Author

ajnart commented May 19, 2024

@manuel-rw @Meierschlumpf we fix conflicts then merge ?

@Meierschlumpf
Copy link
Member

when my part is okay then yes

@ajnart ajnart merged commit f617c60 into dev May 19, 2024
7 checks passed
@Meierschlumpf
Copy link
Member

Please use squash commits instead of merge commits for features 😉

@Meierschlumpf Meierschlumpf deleted the ajnart/fix-duplicate-users branch May 20, 2024 08:40
@Meierschlumpf
Copy link
Member

Please use squash commits instead of merge commits for features 😉

I was able to restrict pull requests to dev to only allow squash

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

Successfully merging this pull request may close these issues.

bug: multiple users with the same name can exist
3 participants