[Cyberduck-trac] [Cyberduck] #684: PRET (PRE Transfer) command

[Cyberduck]

#684: PRET (PRE Transfer) command
-------------------------+--------------------------------------------------
 Reporter:  anonymous    |        Owner:  dkocher 
     Type:  enhancement  |       Status:  reopened
 Priority:  low          |    Milestone:  2.9     
Component:  ftp          |      Version:  2.8.4   
 Severity:  normal       |   Resolution:          
 Keywords:               |  
-------------------------+--------------------------------------------------
Comment (by neothematrix):

 Replying to [comment:5 dkocher]:

 a) yes that's what the specs says, but I've seen some servers not spitting
 out the PRET inside the FEAT command, I think it's better trying to issue
 a PRET command anyway, in the worst case, we just get a "not implemented"
 error, and the connection goes on. Also I saw you have a nice "features()"
 function, but I didn't understand when and if it's called.

 b) from the specs: "Must be sent before the PASV and ONLY the PASV
 command. PRET is unneccessary to be sent before any other command,
 although it can help enable a security feature on clients using Active
 transfers." so, leaving PRET enabled also for Active connections is not a
 bad thing. Anyway, adding a check to see if the connection is passive and,
 only in that case, issue the PRET command is just a matter of add some
 "if", so if you think it's better, I can add this check.

-- 
Ticket URL: <http://trac.cyberduck.ch/ticket/684#comment:6>
Cyberduck <http://cyberduck.ch>
FTP and SFTP Browser for Mac OS X.