Make WordPress Core

Opened 16 years ago

Closed 13 years ago

#16870 closed enhancement (maybelater)

consider disabling HTTP transport after multiple failed requests

Reported by: dd32 Owned by:
Priority: normal Milestone:
Component: HTTP API Version: 2.9
Severity: normal Keywords: 2nd-opinion needs-patch
Cc: Focuses:

Description

There has been a number of reports over the lifespan of the cURL HTTP transport failing with an error during upgrades:

WP HTTP Error: name lookup timed out

This is a common error from cURL on badly configured installations (specifically, IIRC it lacks DNS Resolving in cURL, sometimes it just required an extended amount of time to resolve, others, it simply can't resolve)

Like we do with fsockopen() I think it'd be worth disabling the cURL transport for x hours when it hits this particular error condition..

Related threads: http://wordpress.org/support/topic/dashboard-issues-rss-error-wp-http-error-name-lookup-timed-out and http://wordpress.org/support/topic/plugins-update-failure Both old threads, so this may only affect a PHP4 version of cURL.. For tracking purposes, I've finally created this ticket.

Change History (6)

#1 @hakre
16 years ago

  • Type defect (bug)feature request

Looks like a configuration issue a host and not a defect with wordpress.

Version 0, edited 16 years ago by hakre (next)

#2 @hakre
16 years ago

  • Keywords 2nd-opinion removed

#3 @nacin
16 years ago

  • Keywords 2nd-opinion added
  • Type feature requestenhancement

Restoring dd32's keyword until appropriate feedback is given.

#4 @dd32
15 years ago

  • Summary consider disabling the cURL HTTP transport for failed requestsconsider disabling HTTP transport after multiple failed requests

Looks like a configuration issue on a host and not a defect with wordpress.

Yes, That is the exact problem. The transports are designed that it uses what is available on the host, If WordPress does not take into account a transport failing, we end up in a situation where Updates Notices do not flow through amongst other things. It's in the best interests of WordPress users to protect ourselves against these issues.

#5 @SergeyBiryukov
15 years ago

  • Keywords needs-patch added

There are some similar threads on local forums too.

#6 @dd32
13 years ago

  • Milestone Future Release
  • Resolutionmaybelater
  • Status newclosed

With a dwindling number of transports, and transports which we can increasingly actually rely upon, this isn't as needed as it was 2 years ago.

Note: See TracTickets for help on using tickets.