Pass request arguments to the http_api_debug hook

[johnbillion]

In the HTTP API there's an http_api_debug hook but it doesn't get passed the request arguments. Be nice to get this in to aid a debugging plugin of mine.

[johnbillion]

Patch.

Pretty please for 3.3?

[dd32]

Sounds good to me, there's the http_request_args, pre_http_request and http_response filters available as well, they're useful to catch the start of the request on (and end of a successful request).

If passing extra debug info is of use to plugins, I say do it. I'm curious however, What's the use-case for the plugin?

[johnbillion]

Replying to dd32:

I'm curious however, What's the use-case for the plugin?

The plugin logs HTTP request arguments using http_request_args and the corresponding response using http_response. In order to log the transport, it uses the http_api_debug hook, but this hook doesn't get passed the args so it can't be matched up to the request it belongs to.

[dd32]

In order to log the transport, it uses the http_api_debug hook, but this hook doesn't get passed the args so it can't be matched up to the request it belongs to.

Thought as much, The way I did it in Core Control was to look at the start, and realise that only one request can take place at a time therefor, the next call of the debug hook is the result for it. That being said, I'll throw the patch here in

[johnbillion]

Replying to dd32:

only one request can take place at a time therefor, the next call of the debug hook is the result for it.

Yeah that's correct for the debug hook, but not so for the response hook, which doesn't get fired if the request returns a WP_Error. If a page performs two requests and the first one fails, the first response that fires is for the second request. To account for this my plugin gives each request a hash and adds it to the request args, which can then be used to identify a request and log the error in the dubug hook, or the response in the response hook.

[johnbillion]

And in case you're wondering why I don't just log the responses on the debug hook, it's because an HTTP request caching plugin I use adds some cache data to the response on the response hook, which I also log by hooking onto the response hook with a late priority. (I do like to make things complicated for myself.)

[dd32]

In [18805]:

Pass $args and $url to the http_api_debug action to improve debugging. Props johnbillion. Fixes #18732