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

Integrate Google Captcha to prevent bogus form entries #1117

Closed
kylemh opened this issue May 6, 2020 · 5 comments
Closed

Integrate Google Captcha to prevent bogus form entries #1117

kylemh opened this issue May 6, 2020 · 5 comments
Labels
good first issue If somebody is new to this repository, this is a simple way to dive in. Type: Feature/Redesign

Comments

@kylemh
Copy link
Member

kylemh commented May 6, 2020

Feature request

With #1095 in, I can see more user-facing forms beyond registration on the horizon. I'd rather not wait until they exist to create a system for preventing bot swarms on our site.

Proposed solution

Integrate Google Captcha with this library (the instructions provided by this library are top-notch 👏 )

For now, let's just implement this on pages/join 👍

Here's the client key: 6LcIHfMUAAAAANEzSejRpPilev7Hj0vI9fxc9q9D

Shouldn't be blocked by OperationCode/back-end#331. For now, we can skip the secret token and always approve any recaptcha attempt!

@kylemh kylemh added beginner friendly A relatively new developer should be able to accomplish this task Type: Feature/Redesign good first issue If somebody is new to this repository, this is a simple way to dive in. and removed beginner friendly A relatively new developer should be able to accomplish this task labels May 6, 2020
@Too-Far
Copy link

Too-Far commented May 6, 2020

I'll work on this unless anyone else is dying to do it

@danyalasif
Copy link

@Too-Far Are you working on this? I can take a shot at this for a first issue. :)

@Too-Far
Copy link

Too-Far commented May 8, 2020

I am, I got hung up on something but I think I figured it out.

This was referenced May 8, 2020
@LindaCBeall
Copy link
Contributor

@kylemh, close this out or is more work needed?

@kylemh
Copy link
Member Author

kylemh commented Sep 10, 2020

🤔 I think we can close this. It'd be nice to have rn, but when we move to using Slack OAuth, we wont have a form.

@kylemh kylemh closed this as completed Sep 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue If somebody is new to this repository, this is a simple way to dive in. Type: Feature/Redesign
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants