AFAIK not all the captcha strategies bust the cache, for example recaptcha is cache friendly.

The message about the cache not working in the performance page is scaring some users that actually do not suffer from the problem so I think it should be changed.

Current message:

"The CAPTCHA module will disable the caching of pages that contain a CAPTCHA element."

My proposal:

Most CAPTCHA methods will disable the caching of pages that contain a CAPTCHA element. Check the different implementations to know more about how it affects caching.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rodrigoaguilera created an issue. See original summary.

couturier’s picture

Great idea, and this should be a quick fix. We're hoping to find a new co-maintainer to help with the port to an 8.x stable version.

elachlan’s picture

Status: Active » Needs review
FileSize
816 bytes
elachlan’s picture

I believe the wording needs work. If someone could chime in we can get this done quickly.

couturier’s picture

@elachlan I like the wording in your patch. What about it did you think needed work?

  • elachlan committed 1d21e68 on 8.x-1.x
    Issue #2951324 by elachlan: Add more information to the message about...
elachlan’s picture

Status: Needs review » Fixed
couturier’s picture

Status: Fixed » Needs review

We need to apply this to the new release still, and we should probably have at least one maintainer review it to verify. Then the status will change to Patch to be ported. Once it is ported to the new release, then status may be moved to Fixed. Thanks for your help!

elachlan’s picture

Status: Needs review » Patch (to be ported)

@coururier, It's against dev and I have committed it to 8.x. If you want it in 7.x then I suggest you write a patch to backport it.

couturier’s picture

@elachlan Thanks. No, committing to the 8.x branch only is fine. At this late point past the D8 release, modulels with few development resources like CAPTCHA should be focusing energy on making Drupal 8 modules stable if they aren't already. CAPTCHA used to be really popular in D7, but it still needs a lot of work to be a really reliable module for D8, and the maintainers are currently limited with the time they can devote to it. Thanks for your help.

wundo’s picture

Status: Patch (to be ported) » Fixed
andrey.troeglazov’s picture

Status: Fixed » Closed (fixed)

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