Forward porting of #2449209: Add support for cacheable captcha's (recaptcha) to captcha-8.x-1.x. I am not able to reopen the original issue, so I made this new one.

CommentFileSizeAuthor
#8 cacheable_captcha_support-2974083-8.patch1.64 KBMiroslavBanov
#5 cacheable_captcha_support-2974083-5.patch1.64 KBMiroslavBanov
#3 cacheable_captcha_support-2974083-3.patch1.65 KBMiroslavBanov
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MiroslavBanov created an issue. See original summary.

MiroslavBanov’s picture

MiroslavBanov CreditAttribution: MiroslavBanov as a volunteer and at FFW commented

Recaptcha already is able to cache this correctly, but the message "CAPTCHA session reuse attack detected." cannot be avoided without making a change in this module, even though the submit is otherwise successful, and everything else works. See my comment here: #2219993-31: Enable cacheable captcha support (once 2449209 is committed).

MiroslavBanov’s picture

MiroslavBanov CreditAttribution: MiroslavBanov as a volunteer and at FFW commented
Status: Active » Needs review
FileSize
cacheable_captcha_support-2974083-3.patch1.65 KB

Here's the patch.

Status: Needs review » Needs work

The last submitted patch, 3: cacheable_captcha_support-2974083-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

MiroslavBanov’s picture

MiroslavBanov CreditAttribution: MiroslavBanov as a volunteer and at FFW commented
FileSize
cacheable_captcha_support-2974083-5.patch1.64 KB

Code style fix

MiroslavBanov’s picture

MiroslavBanov CreditAttribution: MiroslavBanov as a volunteer and at FFW commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 5: cacheable_captcha_support-2974083-5.patch, failed testing. View results

MiroslavBanov’s picture

MiroslavBanov CreditAttribution: MiroslavBanov as a volunteer and at FFW commented
Status: Needs work » Needs review
FileSize
cacheable_captcha_support-2974083-8.patch1.64 KB

Trying again.

Status: Needs review » Needs work

The last submitted patch, 8: cacheable_captcha_support-2974083-8.patch, failed testing. View results

Fabianx’s picture

Fabianx CreditAttribution: Fabianx as a volunteer commented
Status: Needs work » Reviewed & tested by the community

RTBC - looks good to me

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 8: cacheable_captcha_support-2974083-8.patch, failed testing. View results

MiroslavBanov’s picture

MiroslavBanov CreditAttribution: MiroslavBanov as a volunteer and at FFW commented

There's something wrong with the build server and the test bot is marking test as failed :)

Mixologic’s picture

Mixologic
He/Him/His
Primary language English
Location Portland, OR
CreditAttribution: Mixologic at Drupal Association commented
Status: Needs work » Reviewed & tested by the community

Testbot is fixed. back to RTBC

couturier’s picture

couturier CreditAttribution: couturier as a volunteer commented
Related issues: +#2982140: Captcha is unusable with current beta and DEV state

This is one of the most active issues in the CAPTCHA queue, so I thought it would be good to post a link here back to the discussion about trying to push for a stable 8.x release of this module so this and other patches could be included. The maintainers are currently looking for some help.

wundo’s picture

wundo CreditAttribution: wundo at Chuva Inc. commented

  • wundo committed 1c2632a on 8.x-1.x authored by MiroslavBanov 
    Issue #2974083 by MiroslavBanov, Fabianx, Mixologic: Port to D8: support...
wundo’s picture

wundo CreditAttribution: wundo at Chuva Inc. commented
Status: Reviewed & tested by the community » Fixed
hass’s picture

hass CreditAttribution: hass commented

@wundo: Can you create a new release soon, please?

wundo’s picture

wundo CreditAttribution: wundo at Chuva Inc. commented

Sure, I will to before the end of this week, I'm planning on reviewing a few more issues before then.

flosuter’s picture

flosuter CreditAttribution: flosuter commented

Any news on the release?
Are there other issues blocking a release?

couturier’s picture

couturier CreditAttribution: couturier as a volunteer commented

@flosuter There has been some activity on this module since a month or so ago, and yes, several patches are still in progress. i don't know when to anticipate a new release but it could easily be another month in my opinion. It could be even longer than that as this module doesn't have a whole lot of attention to it in general as far as port to D8. You might look at the alternative Anti Spam by Clean Talk module for Drupal 8 hosted on drupal.org if you need something right away. It's a highly recommended module and closely maintained and filters spam without requiring a CAPTCHA.

Status: Fixed » Closed (fixed)

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