Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#3 | 2951324-cache-message-3.patch | 816 bytes | elachlan |
|
Comments
Comment #2
couturier CreditAttribution: couturier as a volunteer commentedGreat 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.
Comment #3
elachlan CreditAttribution: elachlan commentedComment #4
elachlan CreditAttribution: elachlan commentedI believe the wording needs work. If someone could chime in we can get this done quickly.
Comment #5
couturier CreditAttribution: couturier as a volunteer commented@elachlan I like the wording in your patch. What about it did you think needed work?
Comment #7
elachlan CreditAttribution: elachlan commentedComment #8
couturier CreditAttribution: couturier as a volunteer commentedWe 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!
Comment #9
elachlan CreditAttribution: elachlan commented@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.
Comment #10
couturier CreditAttribution: couturier as a volunteer commented@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.
Comment #11
wundo CreditAttribution: wundo at Chuva Inc. for Chuva Inc. commentedComment #12
andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commented