Changeset 37694 for trunk/src/wp-includes/class-requests.php

Timestamp:

06/14/2016 05:29:58 AM (9 years ago)

Author:

rmccue

Message:

HTTP API: Update Requests.

This introduces a minimum value of 1 second for timeouts passed to cURL.

Internally, cURL uses alarm() for interrupts, which accepts a second-resolution timeout. Any values lower than 1 second are instantly failed rather than being rounded upwards. While this makes the experience worse for those using asynchronous DNS lookups, there's no way to detect which DNS resolver is being used from PHP.

See #33055, #8923.

File:

• trunk/src/wp-includes/class-requests.php (modified) (1 diff)

trunk/src/wp-includes/class-requests.php

@@ -305 +305 @@
      *
      * - `timeout`: How long should we wait for a response?
+     *    Note: for cURL, a minimum of 1 second applies, as DNS resolution
+     *    operates at second-resolution only.
      *    (float, seconds with a millisecond precision, default: 10, example: 0.01)
      * - `connect_timeout`: How long should we wait while trying to connect?