Backend-side captcha validation with google fails with allow_url_fopen=0 in php.ini because the validation request is sent with file_get_contents.
A patch that fixes the "official" Google ReCaptcha library used in this module will be submitted in comments.

CommentFileSizeAuthor
#4 recaptcha-use_curl-2404083-2.patch2.28 KBcussack
#1 recaptcha-use_curl-2404083-1.patch2.32 KBcussack

Comments

cussack’s picture

cussack commented
StatusFileSize
new recaptcha-use_curl-2404083-1.patch2.32 KB

attach proposed patch.

cussack’s picture

cussack commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: recaptcha-use_curl-2404083-1.patch, failed testing.

cussack’s picture

cussack commented
StatusFileSize
new recaptcha-use_curl-2404083-2.patch2.28 KB

fix patch to be appliable on repository.

cussack’s picture

cussack commented
Status: Needs work » Needs review

  • diolan committed ea71857 on 7.x-2.x 
    Issue #2404083 by christoph-eatsmarter: Applied and tested patch by...
diolan’s picture

diolan commented
Status: Needs review » Fixed

Your patch is applied to the 7.x-2.x-dev version. Thank you!

Status: Fixed » Closed (fixed)

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

hass’s picture

hass commented

cURL support may be added soon to the lib, see https://github.com/google/recaptcha/pull/39

hass’s picture

hass commented
Parent issue: » #2473141: Add cURL support
hass’s picture

hass commented

The cURL support will be dropped by #2472811: Upgrade lib to latest version. Please help on the lib case https://github.com/google/recaptcha/pull/39 to get this feature in so we can update the library asap and get this back.

cussack’s picture

cussack commented

I contacted that guy and wrote a test for his code, let's see what happens.

hass’s picture

hass commented

Great. Thanks for info. Have you tested if the workaround #2473605: Add workaround for locked down PHP servers (allow_url_fopen = Off) may also work for you?

cussack’s picture

cussack commented

Not yet, but I will do that soon.
Do you have any hints on how to speed up the merge of the pull request? Or is it simply the CLA that's missing?

hass’s picture

hass commented

I guess it is the CLA... not sure if he completed this.

hass’s picture

hass commented

@cussack: You may also like to join #2481161: Implement RequestMethod based on drupal_http_request() if drupal_http_request works for your this should also work for your. Maybe you can just help testing this solution.

cussack’s picture

cussack commented

#2481161: Implement RequestMethod based on drupal_http_request() works for me, many thanks