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

Signup duplication #6988

Closed
1 task
fakkio opened this issue Nov 30, 2022 · 2 comments
Closed
1 task

Signup duplication #6988

fakkio opened this issue Nov 30, 2022 · 2 comments
Labels
bug/needs-info More information is needed for reproduction

Comments

@fakkio
Copy link
Contributor

fakkio commented Nov 30, 2022

What's not working?

Hi 👋🏼,

during signup with authdb authentication, rw checks for duplication in the username (i.e. email), but I think, the username check should be case insensitive.

How do we reproduce the bug?

GitPod snapshot: https://gitpod.io#snapshot/61447fb9-8e28-4093-a1ee-b7f0dc65f6e2

try to sigup with email@example.com and then with Email@example.com. You can signup without any problem.

What's your environment? (If it applies)

No response

Are you interested in working on this?

  • I'm interested in working on this
@fakkio fakkio added the bug/needs-info More information is needed for reproduction label Nov 30, 2022
@redwoodjs-bot redwoodjs-bot bot added this to Main Nov 30, 2022
@redwoodjs-bot redwoodjs-bot bot moved this to Triage in Main Nov 30, 2022
@ageddesi
Copy link
Contributor

I would be interested in working on this by adding .toLowerCase on the checks.

@thedavidprice
Copy link
Contributor

This is effectively a duplicate of #7787 At the very least it would be best to combine efforts. @fakkio do take a look at some suggestions there as well.

@ageddesi could you join the conversation on #7787 and offer thoughts/help based on the discussion so far? @ched-dev is looking for feedback/suggestions. And both Rob and Tobbe are out this week.

Zero pressure either way. Closing this out.

@github-project-automation github-project-automation bot moved this from Triage to Done in Main Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/needs-info More information is needed for reproduction
Projects
Status: Done
Development

No branches or pull requests

3 participants