[wp-trac] [WordPress Trac] #10522: timeout for ftp connections as a constant in config.php

WordPress Trac wp-trac at lists.automattic.com
Fri Aug 7 09:03:39 UTC 2009


#10522: timeout for ftp connections as a constant in config.php
--------------------------+-------------------------------------------------
 Reporter:  ntm           |       Owner:  dd32                   
     Type:  defect (bug)  |      Status:  accepted               
 Priority:  normal        |   Milestone:  2.9                    
Component:  Filesystem    |     Version:  2.8.2                  
 Severity:  normal        |    Keywords:  has-patch needs-testing
--------------------------+-------------------------------------------------
Changes (by dd32):

  * keywords:  ftp connection timeout, plugin upgrade, => has-patch needs-
               testing


Comment:

 > attachment 10522.diff added
  * Adds
   * FTP_CONNECT_TIMEOUT - A Timeout for the initial connecting 30s
   * FTP_TIMEOUT
    * Used by SSH/ftpsockets as a per-action timeout 30s
    * Used by ftpext as a per-connection timeout, Set appropriately higher
 in that class 240s (4min)
  * no sanitization is done on the constants, If a user defines them, they
 need to make sure its set to a value which suits their system. IMO, the
 defaults i've selected should be more than enough for most people.
  * ftpsockets was previously defaulting to 30s for the connect/action
 timeouts.

 ...And the patch is -completely- untested, It may even have fatal PHP
 errors, but syntax looks sane to me.

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


More information about the wp-trac mailing list