Skip to content
This repository has been archived by the owner on Aug 17, 2021. It is now read-only.

Use recaptcha.net where google is not available. #234

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mihaisavezi
Copy link

Specifically, China.

More info here: https://developers.google.com/recaptcha/docs/faq

"Yes, please use "www.recaptcha.net" in your code in circumstances when "www.google.com" is not accessible"

refers to google/recaptcha#87.

According to "https://developers.google.com/recaptcha/docs/faq"

"Can I use reCAPTCHA globally?
Yes, please use "www.recaptcha.net" in your code in circumstances when "www.google.com" is not accessible.

First, replace <script src="https://www.google.com/recaptcha/api.js" async defer></script> with <script src="https://www.recaptcha.net/recaptcha/api.js" async defer></script>
After that, apply the same to everywhere else that uses "www.google.com/recaptcha/" on your site."
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 94.406% when pulling 1b99dad on mihaisavezi:master into 73d01e4 on VividCortex:master.

@christianesperar
Copy link

Any new about this PR? We have the same requirements and this solution can be a fix.

@mtrias
Copy link
Contributor

mtrias commented Oct 2, 2019

Apologies, we missed this. So, Code looks good to be merged, but it's missing tests, and we try not to merge untested code. Can someone add a couple unit tests so this can be merged?

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

Successfully merging this pull request may close these issues.

4 participants