wp-media / wp-rocket

Performance optimization plugin for WordPress
https://wp-rocket.me
GNU General Public License v2.0
687 stars 215 forks source link

Prewarmup home in certain scenarios where it shouldn't #6850

Open Mai-Saad opened 1 month ago

Mai-Saad commented 1 month ago

Before submitting an issue please check that you’ve completed the following steps:

Describe the bug Home page prewarmup occurs when switch theme/change permalinks and clear critical images for license expired > 15d

To Reproduce Steps to reproduce the behavior:

  1. license expired > 15d
  2. fresh install ===> no warmup
  3. clear critical images
  4. check WPR log and ATF table ==> home page added to log and ATF table in DB

Expected behavior Prewarmup shouldn't work for license > 15

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

Acceptance Criteria (for WP Media team use only) Updated intervals which was expected based on discussion with PO before

Tabrisrp commented 3 weeks ago

Reproduce the issue ✅

Yes

Identify the root cause ✅

There is no check for the grace period in the method handling the homepage warmup

Scope a solution ✅

in Common\PerformanceHints\WarmUp\Controller

Estimate the effort ✅

Effort S