WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#3765 closed defect (bug) (fixed)

Cron creates infinite Loop when using Wordpress with SSL

Reported by: dragonhard Owned by: rob1n
Milestone: 2.2 Priority: high
Severity: major Version: 2.1
Component: General Keywords: has-patch commit
Focuses: Cc:

Description

In Wordpress 2.1 the Cron.php calles every second itself and blockes the server. This only happens when using the Wordpress-Blog with ssl under https. The infinite Loop causes serious problems in the server performance. Problem appeared since Wordpress 2.1.

Attachments (3)

cron.php.diff (1.3 KB) - added by dragonhard 7 years ago.
Patch for infinite SSL-Loop in Wordpress 2.1
3765.diff (885 bytes) - added by rob1n 7 years ago.
3765.2.diff (641 bytes) - added by rob1n 7 years ago.

Download all attachments as: .zip

Change History (17)

dragonhard7 years ago

Patch for infinite SSL-Loop in Wordpress 2.1

comment:1 filosofo7 years ago

Could you be missing wp-cron.php?

See this: #3713

comment:2 dragonhard7 years ago

No. wp-cron.php is not missing....

comment:3 Nazgul7 years ago

  • Keywords has-patch added
  • Milestone changed from 2.1.1 to 2.1.2

2.1.1 is out, so bumping the milestone to 2.1.2.

comment:4 rob1n7 years ago

  • Owner changed from anonymous to rob1n
  • Status changed from new to assigned

While the patch looks sound, I think the if/else's are a little weird. I'll work on a new patch.

rob1n7 years ago

comment:5 rob1n7 years ago

  • Keywords ssl removed

comment:6 ryan7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [4980]) Fix spawning with SSL. Props dragonhard and rob1n. fixes #3765

comment:7 foolswisdom7 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Robert Deaton to wp-hackers 7:04 pm:
This needs a check to make sure OpenSSL support is compiled in.
Checking for the existence of one of the functions at
http://php.net/openssl should do.

comment:8 rob1n7 years ago

  • Keywords has-patch removed

I'm on it.

rob1n7 years ago

comment:9 rob1n7 years ago

  • Keywords has-patch commit added

Added a check for function_exists('openssl_open').

comment:10 rob1n7 years ago

  • Milestone changed from 2.1.3 to 2.2

comment:11 rob1n7 years ago

Fools: 2.1.x or 2.2?

comment:12 rob1n7 years ago

  • Status changed from reopened to new

comment:13 rob1n7 years ago

  • Status changed from new to assigned

comment:14 ryan7 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [4984]) Make sure openssl is available. Props Robert Deaton and rob1n. fixes #3765

Note: See TracTickets for help on using tickets.