-
-
Notifications
You must be signed in to change notification settings - Fork 139
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
ADD ReCaptcha feature for anonymous User #682
Conversation
bugheist/settings.py
Outdated
@@ -386,3 +386,6 @@ | |||
STRIPE_TEST_PUBLIC_KEY = os.environ.get("STRIPE_TEST_PUBLIC_KEY", "pk_test_51HFiXMFf0OkkOVnDkNs4opFLqM0Sx5GA6Pedf63uGzG1gHhumFYHEOLfCA7yzZwXUpjaa5j9ZhS1yciNhouYCMh400pSx5ZEx6") | |||
STRIPE_TEST_SECRET_KEY = os.environ.get("STRIPE_TEST_SECRET_KEY", "sk_test_51HFiXMFf0OkkOVnDiAnuYiq6JInx3VSXw2HzIV6ihGWzaO8un5djIi990OCLTAv5PRnY7Yl8v8yuxf6yU47gvKYj00hkKaKaQ0") | |||
STRIPE_LIVE_MODE = False # Change to True in production | |||
|
|||
GOOGLE_RECAPTCHA_SECRET_KEY = env('GOOGLE_RECAPTCHA_SECRET_KEY', default="6LeV4GwaAAAAAC3AluQ3i3GgAv7XHA2EhK9sAkgi") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these dummy values? If not, could you replace it with something like dummy
?
@ankit2001 - Left a comment, otherwise LGTM! |
@souravbadami I have done the required changes, please review it, also please register on the link i provided in the first comment and add site and secret key to settings.py to make it work. |
LGTM! Thanks @ankit2001. |
ADD ReCaptcha feature for anonymous User
This PR aims to add captcha feature and fixing the issue #681 .
You can make your own secret and site key for captcha by registering on https://www.google.com/recaptcha/admin/create
and add them in .env file or replace with the keys described in settings.py, also i have added default key for testing.
Below are the some results of the added feature.
If Captcha is not configured.
The output of request
If Captcha is Configured.
The output of request
Please review it @fredfalcon , @souravbadami , @srahulbadami and suggest me changes if needed.
Thanks.