[wp-trac] [WordPress Trac] #10407: FTP Extension-based filesystem fails on WP core upgrade/re-install

WordPress Trac wp-trac at lists.automattic.com
Thu Jul 16 00:19:35 UTC 2009


#10407: FTP Extension-based filesystem fails on WP core upgrade/re-install
-------------------------------+--------------------------------------------
 Reporter:  Denis-de-Bernardy  |       Owner:  dd32
     Type:  defect (bug)       |      Status:  new 
 Priority:  normal             |   Milestone:  2.9 
Component:  Filesystem         |     Version:      
 Severity:  normal             |    Keywords:      
-------------------------------+--------------------------------------------

Comment(by Denis-de-Bernardy):

 Replying to [comment:8 dd32]:
 > > It's actually not. It creates bugs in apply_filters() and do_action(),
 which were closed (as wontfix, for good reasons) when I reported them.
 >
 > Completely different scenario's. remove_filter will work in that case,
 Its only an issue when removing functions hooked to the current filter
 that is not the current function. (ie. As long as you're not attempting to
 remove a function that is going to be fired on THIS iteration of the
 stack)

 No no, it also affects later functions on the same hook in some weird edge
 cases. But that's not the point of this ticket, is it? :-)

 > > but based on the workaround code I had to use when I was maintaining
 my own FTP filesystem class (i.e. disconnect/reconnect after so many
 files), it actually is the entire sessions' timeout on a lot of platforms.
 >
 > Which sounds entirely possible, Just need to find a value which is going
 to work for all of core.. I know FTP takes sub 5 seconds in total on my
 new host.. I know it takes >120seconds on others.

 It took well over 90s on Jeff's host. I've bumped it to 600s (as I did for
 the timeout) to see how things went. Will report back when he replies.

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


More information about the wp-trac mailing list