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

Prevent spam users from registering #155

Open
ColeDCrawford opened this issue Mar 25, 2021 · 8 comments · May be fixed by #160
Open

Prevent spam users from registering #155

ColeDCrawford opened this issue Mar 25, 2021 · 8 comments · May be fixed by #160

Comments

@ColeDCrawford
Copy link
Contributor

We have several hundred spam users on dev since open signups were allowed.

@ColeDCrawford
Copy link
Contributor Author

Screen Shot 2021-03-25 at 10 57 45 AM

@rsinghal
Copy link
Contributor

@lukehollis This is high priority - it should be addressed ASAP or we will need to disable signups.

@ColeDCrawford
Copy link
Contributor Author

We are up to ~700 spam users.

@rsinghal
Copy link
Contributor

@npicardo @lukehollis We are going to have to disable signups until this gets resolved. I also see that it allows for the same email address to be registered for different usernames, which should also be fixed.

@lukehollis
Copy link
Contributor

To address this, I'll try integrating the https://pypi.org/project/django-recaptcha/ package unless anyone has another preference.

@lukehollis lukehollis linked a pull request Apr 15, 2021 that will close this issue
@lukehollis
Copy link
Contributor

Addressed in #160 -- to configure, now the secure.py.example has support for recaptcha fields:

Screen Shot 2021-04-15 at 8 29 45 AM

@lukehollis
Copy link
Contributor

Additionally, you may need to add your dev domains on your local environment to your keys.

@lukehollis lukehollis removed their assignment Jul 8, 2021
@rsinghal
Copy link
Contributor

Not working in the PR

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

Successfully merging a pull request may close this issue.

3 participants