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

Subscription block stops caching for admin users #38152

Open
donnchawp opened this issue Jul 2, 2024 · 0 comments
Open

Subscription block stops caching for admin users #38152

donnchawp opened this issue Jul 2, 2024 · 0 comments
Labels
[Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Plugin] Super Cache A fast caching plugin for WordPress. [Pri] Low Triaged [Type] Bug When a feature is broken and / or not performing as intended

Comments

@donnchawp
Copy link
Contributor

Impacted plugin

Jetpack

Quick summary

The Subscriptions block at extensions/blocks/subscriptions/subscriptions.php sets DONOTCACHEPAGE on line 1033 when an admin user is logged in regardless of whether a subscription block is being shown on the current page.

ref: p1707741294383739/1707737397.873219-slack-C016BBAFHHS

Steps to reproduce

  1. Enable the newsletter feature in Jetpack.
  2. In WP Super Cache enable caching for all
  3. Enable debugging in WPSC
  4. Visit the blog as an admin user and note that DONOTCACHEPAGE is set so that no caching is done.

A clear and concise description of what you expected to happen.

The page should be cached as WP Super Cache is configured to allow it.

What actually happened

The page was not cached.

Impact

Some (< 50%)

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

No response

Logs or notes

Instead of checking for the admin user, it should probably check for the subscription cookie, I think. I haven't used the premium features of this block and I'm not 100% certain how it works.

@donnchawp donnchawp added [Type] Bug When a feature is broken and / or not performing as intended [Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. Needs triage Ticket needs to be triaged labels Jul 2, 2024
@github-actions github-actions bot added [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] Low labels Jul 2, 2024
@darssen darssen added the [Plugin] Super Cache A fast caching plugin for WordPress. label Jul 2, 2024
@jeherve jeherve added Triaged and removed Needs triage Ticket needs to be triaged labels Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Subscriptions All subscription-related things such as paid and unpaid, user management, and newsletter settings. [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Plugin] Super Cache A fast caching plugin for WordPress. [Pri] Low Triaged [Type] Bug When a feature is broken and / or not performing as intended
Projects
Development

No branches or pull requests

3 participants