[wp-trac] [WordPress Trac] #20074: Attempt MD5 checksum verification in upgrader if available

[WordPress Trac]

#20074: Attempt MD5 checksum verification in upgrader if available
------------------------------------+------------------
 Reporter:  sivel                   |       Owner:
     Type:  enhancement             |      Status:  new
 Priority:  normal                  |   Milestone:  3.4
Component:  Upgrade/Install         |     Version:  2.8
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------

Comment (by westi):

 [attachment:20074.2.diff] is my preferred of the later two patches but I'm
 not a big fan of extending download_url to optionally return the HTTP
 Repsonse object.

 The whole point of the function is you either get a temp filename where
 the file has been downloaded to or an Error object adding a third thing
 into the mix complicates things for the users of the function
 unnecessarily - I think if you want access to the full HTTP response
 object you should make your own version of the function.

 Also the current patch leaves files lying around because it is the
 responsibility of the caller of download_url to unlink but if the funtion
 returns the full response you don't have access to the temp file to
 unlink.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/20074#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software