Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Overcautious failure to cache when nonce in markup #645

Closed
KTS915 opened this issue Dec 27, 2015 · 3 comments
Closed

Bug: Overcautious failure to cache when nonce in markup #645

KTS915 opened this issue Dec 27, 2015 · 3 comments

Comments

@KTS915
Copy link

KTS915 commented Dec 27, 2015

This is related to #642, but Akismet is not involved.

ZenCache Pro v151220 apparently now refuses to cache any page (at least for logged-in users) where some mention is made of nonce. This is pretty serious because it means effectively that I can't cache any pages on some of my sites.

Plugins that include nonce in the markup include those, like BAW Login/Logout menu, that provide a means of logging in and out, and plugins that handle comments. Obviously, this is a pretty wide-ranging issue.

I can also see that ZenCache Pro v151220 itself adds nonce to the markup, though I don't know whether this is contributing to the problem.

I have reverted to v151105 for the moment, and have no problems. I notice that v151105 does not appear to add nonce to the markup.

@KTS915 KTS915 changed the title Bug: Overcautious failure to cahce when nonce in markup Bug: Overcautious failure to cache when nonce in markup Dec 27, 2015
@raamdev
Copy link
Contributor

raamdev commented Dec 27, 2015

@KTS915 Thanks for the report.

ZenCache should be automatically disabling the Akismet nonce (which is supported by Akismet for scenarios such as when caching pages), however a bug in the way Akismet reads the value of the filtered option is preventing ZenCache v15120 from working as expected (see #642; we have a temporary workaround posted in that issue too if you're using Akismet).

We also have a feature request open to make it possible to disable the nonce exclusion; see #637. It sounds like that's what you need--a way of telling ZenCache to ignore nonce values in the HTML and cache the page anyway.

ZenCache never adds a nonce to any front-end pages; it only uses them on the Dashboard when generating links for the options pages. Since ZenCache only caches the front-end of the site, the nonce values that ZenCache adds have no effect.

Caching nonce values is a bad idea, since the nonce is time-sensitive. That's why we made the decision to have ZenCache ignore HTML that contains any nonce values. Please see the original issue, #601, for full details.


I'm going to close this issue in favor of #637, which shold resolve this issue for you.

@KTS915
Copy link
Author

KTS915 commented Dec 27, 2015

OK, thanks, Raam. I don't use Akismet on any sites. I look forward to finding out how to disable the nonce exclusion.

@raamdev
Copy link
Contributor

raamdev commented Jan 3, 2016

ZenCache Pro v160103 has been released and includes changes from this GitHub Issue. See the v160103 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference #637.

@wpsharks wpsharks locked and limited conversation to collaborators Jan 3, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants