Make WordPress Core

Opened 10 years ago

Closed 4 years ago

Last modified 3 months ago

#27722 closed defect (bug) (maybelater)

Don't fire a warning when the WP API timeouts — or do so properly.

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by: johnbillion's profile johnbillion
Milestone: Priority: normal
Severity: normal Version: 3.7.1
Component: Upgrade/Install Keywords: needs-patch
Focuses: Cc:

Description

At the time of writing, I'm systematically getting the following error from within the WP admin area's update core section:

An unexpected error occurred. Something may be wrong with WordPress.org or this server's configuration.

Two things bother me:

  1. It only fires when SSL is used.
  2. The raw_response content suggests a connection timeout.

I think the warning itself is of dubious merit. If we decide to keep it, it should be more explicit as to what is going on. And it should always fire, regardless of whether SSL is getting used or not…

A possible exception I'd make is when it's a request timeout — in that case WP should silently try again a few minutes later — without a warning or notice.

Change History (13)

#1 @ocean90
10 years ago

I have noticed this as well some time ago. Only SSL connections are failing sometimes with Operation timed out after 3000 milliseconds with 0 bytes received.

If we decide to keep it, it should be more explicit as to what is going on.

The notice is triggered as E_USER_WARNING or E_USER_NOTICE and are for regular users not visible. When the SSL request fails it will fallback to a non SSL request, see #25716.

#2 follow-up: @SergeyBiryukov
10 years ago

I get this warning in the admin without SSL too. Sounds like a duplicate of #25788 and #27091.

#3 @Denis-de-Bernardy
10 years ago

Another nice to have feature would be to make these checks run on a cron job, rather than during a page load. When you've a bad internet connection, it prevents from loading.

#4 in reply to: ↑ 2 @ocean90
10 years ago

Replying to SergeyBiryukov:

I get this warning in the admin without SSL too. Sounds like a duplicate of #25788 and #27091.

Yeah, thanks for the hint. I identified that my code for excluding wp.com themes from updates was broken. Works now fine.

#5 @johnbillion
10 years ago

  • Keywords dev-feedback added
  • Version changed from trunk to 3.7.1

#6 @SergeyBiryukov
9 years ago

#33197 was marked as a duplicate.

#7 @atomicjack
9 years ago

I'm receiving this error all the time in WordPress 4.3-RC. Can we look into a fix?

#8 @johnbillion
9 years ago

  • Keywords needs-patch added; dev-feedback removed
  • Milestone changed from Awaiting Review to 4.4
  • Owner set to johnbillion
  • Status changed from new to accepted

This drives me nuts.

#9 @NickDuncan
9 years ago

This could also be triggered as a result of plugins making use of the "plugins_api" filter to modify where their plugins' are updated from. i.e. doing a wp_remote_post to their own server instead of WordPress.org and the result being a timed out connection or an unknown response, thereby resulting in the error being thrown.

I suppose we need to understand why the error is being shown in the first place and if it is in fact helpful.

I believe our options are as follows:

  1. Simply do not show the error
  2. Show the error, but give more detail so they user can make an informed decision
  3. Show the error with more detail but perhaps show it in the normal <div class='error'> notice instead of the traditional WP_Error output.

Can we get a consensus on what we all feel is the best option? I can then try come up with a patch.

#10 @johnbillion
9 years ago

  • Milestone changed from 4.4 to Future Release

#11 @GregLone
8 years ago

So. Anybody to give this ticket some love?

#12 @johnbillion
4 years ago

  • Resolution set to maybelater
  • Status changed from accepted to closed

Closing due to lack of interest.

#13 @dd32
3 months ago

#61166 was marked as a duplicate.

Note: See TracTickets for help on using tickets.