[wp-trac] Re: [WordPress Trac] #2933: Show messages from weblogUpdates.ping/extendedPing

[WordPress Trac]

#2933: Show messages from weblogUpdates.ping/extendedPing
-------------------------------------------------+--------------------------
 Reporter:  c960657                              |        Owner:  anonymous  
     Type:  enhancement                          |       Status:  new        
 Priority:  normal                               |    Milestone:  2.3 (trunk)
Component:  XML-RPC                              |      Version:             
 Severity:  normal                               |   Resolution:             
 Keywords:  has-patch 2nd-opinion needs-testing  |  
-------------------------------------------------+--------------------------
Comment (by c960657):

 The updated patch, ping-4.patch, now uses HTTP status codes for removal.

 If the server returns 403 Forbidden, 404 Not found or 410 Gone, and this
 is the third failure in a row, the server is removed. If other errors
 happen, the server is removed after 20 failures in a row.

 Also, the client no longer tries to call weblogUpdates.ping, if the call
 to weblogUpdates.extendedPing failed, unless the first call returned a
 "200 OK". If the server returns anything else, it is an indication that
 the URL does not point to an XML-RPC service (according to the XML-RPC
 spec, servers should always return 200 OK). This will also ease the load
 on servers that are rejecting pings.

 I like your suggestion about "301 Moved permanently", but my patch doesn't
 address that. Perhaps this can be implemented in another issue.

 Comments are welcome. I hope people will try out the patch and see if it
 works for them.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/2933#comment:13>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software