[wp-trac] Re: [WordPress Trac] #8726: cron implementation uses POST
method instead of GET method
WordPress Trac
wp-trac at lists.automattic.com
Thu Jan 15 23:17:04 GMT 2009
#8726: cron implementation uses POST method instead of GET method
-------------------------------------------------------+--------------------
Reporter: patpro | Owner: anonymous
Type: defect (bug) | Status: new
Priority: normal | Milestone:
Component: Administration | Version: 2.7
Severity: major | Resolution:
Keywords: has-patch commit 2nd-opinion dev-feedback |
-------------------------------------------------------+--------------------
Comment (by jacobsantos):
Yeah, no, HTTP authentication support is in theory supported, but only if
you, the plugin author, send the correct Headers yourself. The !WordPress
HTTP API will not automatically send the correct headers, or use the
correct options for cURL and HTTP extension APIs.
So, yes, it could do HTTP authentication, but a patch would need to be
made to support it and it wouldn't look pretty unless the HTTP API had
native support for it.
I think the problem is more or less, that the behavior has changed and it
shouldn't have changed. I think I originally used POST instead of GET,
because the API is not retrieving anything.
I think it is a WTF that the server has such requirements for POST
requests, but the change was unexpected for the user, so the patch was
made to put the behavior back to the original. If it is affecting this
user who reported the ticket, then it might be affecting more people.
If it isn't affecting more people, then yes, the setup the user has is
fringe and isn't worth supporting. Well, if the change was major, but it
wasn't. If it will fix the problem for the reporter with a simple change
that doesn't require the user to keep making the same change for each
upgrade, then I say commit it.
--
Ticket URL: <http://trac.wordpress.org/ticket/8726#comment:3>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list