Currently we have customers in China that are unable to submit web forms with reCAPTCHA on them.

According to Google the fix is to change the domain name of the api to www.recaptcha.net.

See FAQs here https://developers.google.com/recaptcha/docs/faq

Would it be possible to build in an option to the reCAPTCHA module to switch the domain if needed.

CommentFileSizeAuthor
#20 2993365-update-verify-url-use-globally.patch918 bytesFrank Yu
#19 2993365-update-verify-url-use-globally.patch919 bytesFrank Yu
#14 Issue-2993365-by-Prashantc-blakehall-hass-vmilic-Add.patch5.3 KBhass
#10 use-recaptcha-globally-2993365-10.patch4.58 KBblakehall
#4 use-recaptcha-globally-admin-setting.png41.23 KBprashant.c
#3 use-recaptcha-globally-2993365-3.patch5.15 KBprashant.c

Comments

simonmc created an issue. See original summary.

hass’s picture

hass commented
Version: 7.x-2.x-dev » 8.x-2.x-dev

We can do this.

prashant.c’s picture

prashant.c
he/him
Primary language English
Location Dharamshala
commented
Status: Active » Needs review
StatusFileSize
new use-recaptcha-globally-2993365-3.patch5.15 KB

Tried my hands to implement this feature.
Submitting patch for the same. Patch needs to be reviewed.

Thanks.

prashant.c’s picture

prashant.c
he/him
Primary language English
Location Dharamshala
commented
StatusFileSize
new use-recaptcha-globally-admin-setting.png41.23 KB

In the above patch i have added an option in the admin settings of the recaptcha to enable "Use reCAPTCHA globally" which will switch the URLs in the script based on this setting.

  • hass committed 4250ffe on 8.x-2.x authored by Prashant.c 
    Issue #2993365 by Prashant.c, hass: Add an option to use reCAPTCHA...
hass’s picture

hass commented
Status: Needs review » Fixed
hass’s picture

hass commented
Version: 8.x-2.x-dev » 7.x-2.x-dev
Status: Fixed » Needs work

Can you backport to D7, please? I made small changes to your patch, so we need to use the code commited for backporting.

hass’s picture

hass commented
Status: Needs work » Patch (to be ported)
msantos202’s picture

msantos202 commented

Hi,
Sorry, but where is the patch for Drupal 7 I can not find it?
I change to dev version 7.x-2.x-dev but this patch is not applied to this versions.

Thanks,

blakehall’s picture

blakehall commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new use-recaptcha-globally-2993365-10.patch4.58 KB

Here is a quick patch for the Drupal 7 version.

eojthebrave’s picture

eojthebrave
he/him
Primary language English
Location Minneapolis, MN
commented
Status: Needs review » Reviewed & tested by the community

D7 backport looks good to me. Thanks @blakehall.

hass’s picture

hass commented
Status: Reviewed & tested by the community » Needs work
  • Variable uninstall is missing.
  • The D8 translatable strings are not reused. Please sync them with my commit and not tha patch attached to the case as noted above.
  • Tests are missing.
  • Migration of the variable is missing.
vmilic’s picture

vmilic commented

Has anyone tried to use it from China/Russia? It looks like the recaptcha.net workaround doesn't really work:

hass’s picture

hass commented
Status: Needs work » Needs review
StatusFileSize
new Issue-2993365-by-Prashantc-blakehall-hass-vmilic-Add.patch5.3 KB

  • hass committed 7b0370e on 7.x-2.x 
    Issue #2993365 by Prashant.c, blakehall, hass, vmilic: Add an option to...
hass’s picture

hass commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

GUE-Andreas’s picture

GUE-Andreas commented

With the last patch in place, I'm still getting reports that it doesn't work from China.

Any others?

Edit: Apparently it IS working after all. So, never mind. ;)

Frank Yu’s picture

Frank Yu commented
Version: 7.x-2.x-dev » 8.x-3.0
StatusFileSize
new 2993365-update-verify-url-use-globally.patch919 bytes

I think also need to update verify url use globally

Frank Yu’s picture

Frank Yu commented
StatusFileSize
new 2993365-update-verify-url-use-globally.patch918 bytes

Update comment