WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 22 months ago

#20537 new defect (bug)

Don't spawn cron requests for suspended blogs

Reported by: ryan Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.3.1
Component: Cron API Keywords: has-patch 2nd-opinion
Focuses: Cc:

Description (last modified by ryan)

For multisite, spawning cron requests is wasteful and unnecessary. wp_cron()/spawn_cron()/wp-cron.php should check the blog's status and bail if the blog is suspended.

Attachments (2)

20537.diff (481 bytes) - added by ryan 2 years ago.
20537.2.patch (838 bytes) - added by kurtpayne 2 years ago.
Alternate patch

Download all attachments as: .zip

Change History (8)

comment:1 ryan2 years ago

Cribbing from ms_site_check(), the conditions to check:

if ( '1' == $current_blog->deleted || '2' == $current_blog->deleted || '1' == $current_blog->archived || '1' == $current_blog->spam )
 return;

Or from get_blogs_of_user():

if ( $blog->archived || $blog->spam || $blog->deleted )
     return;
Last edited 2 years ago by ryan (previous) (diff)

ryan2 years ago

comment:2 nacin2 years ago

I think spawning is unnecessary. wp-cron.php should still be able to be triggered, though. Someone somewhere is surely doing something weird with regards to this.

I think it would be best if bailed early in spawn_cron() and didn't hook wp_cron() directly into sanitize_comment_cookies. That would allow for wp_cron() and wp-cron.php to still execute.

I could also just be overly careful.

comment:3 follow-up: ryan2 years ago

  • Description modified (diff)

I can see it being good and bad, at least when considering core events. A hard freeze prevents future posts from accidentally getting published and pingbacks from accidentally going out from a blog that should be dead. The blog and the cron queue are frozen. Then again, there could be some events that should still be processed. This was requested by some wp.com folks to shut up zombie blogs. How dead is dead enough?

comment:4 ryan2 years ago

And, yes, spawning should be shut down and the hook avoided. Finishing the patch tomorrow.

kurtpayne2 years ago

Alternate patch

comment:5 kurtpayne2 years ago

  • Cc kpayne@… added
  • Keywords has-patch 2nd-opinion added

20537.2.patch is the same as 20537.diff, but in a different place. It prevents the cron request from spawning for zombie blogs, but still allows wp-cron.php to be called directly.

Still a bit new to the cron code. Is this right?

comment:6 in reply to: ↑ 3 nacin22 months ago

Replying to ryan:

How dead is dead enough?

I'm convinced. Both spawn_cron() and wp-cron should bail. Maybe a filter in wp-cron for the creative.

Note: See TracTickets for help on using tickets.