4udak / pyftpdlib

Automatically exported from code.google.com/p/pyftpdlib
Other
1 stars 1 forks source link

Close data connection before 226 Reply #156

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
rfc959 says in chapter 5.2.  CONNECTIONS:

..."To prevent a race condition here, the server
      sends a reply (226) after closing the data connection".

DTPHandler.handle_close works the other way round, it first sends the 226 
Reply, then closes the data connection.

>What steps will reproduce the problem?
1.Look at implementation,  
2. or make a wireshark of an uploadand inspect it

>What version of pyftpdlib are you using? On what operating system? >Which
>Python version?
pyftpdlib 0.5.2, but 0.6.0 has identical implementation.

Most ftp clients obviously have no problem with this implementation, but I have 
do deal with a ftp client in an embedded device that can't handle this. :(
It is easy to fix for me, but pyftpdlib should implement the rfc correctly.

Original issue reported on code.google.com by bernd.de...@gmail.com on 3 Mar 2011 at 6:22

GoogleCodeExporter commented 9 years ago
Thanks. The fix is not easy as it might seem though. I'll have to think about 
it carefully.

Original comment by g.rodola on 3 Mar 2011 at 7:43

GoogleCodeExporter commented 9 years ago
This should now be fixed in r837.
Please let me know if it solves your problem.

Original comment by g.rodola on 3 Mar 2011 at 8:10

GoogleCodeExporter commented 9 years ago
Works fine, thanks!

Original comment by bernd.de...@gmail.com on 8 Mar 2011 at 10:03

GoogleCodeExporter commented 9 years ago

Original comment by g.rodola on 18 Mar 2011 at 10:43

GoogleCodeExporter commented 9 years ago

Original comment by g.rodola on 3 Jan 2012 at 11:40

GoogleCodeExporter commented 9 years ago
0.7.0 is out. Closing this out as definitively fixed.

Original comment by g.rodola on 25 Jan 2012 at 7:24