Update document_ftp pyftpdlib version

Bug #1115159 reported by YannickB (YOLO consulting)
24
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Confirmed
Wishlist
OpenERP's Framework R&D

Bug Description

Hi guys,

Working on document_ftp module, I realized that the pyftpdlib http://code.google.com/p/pyftpdlib/ used by OpenERP is out of date since 2008, event in V7. Three major version of the binary was missed.

We are using document_ftp in a very intensive ftp project and we have some problem which are corrected in the latest version of pyftpdlib. We are managing theses problems, but for others projects I strongly suggest to make the version upgrade pyftpdlib 0.4 to 0.7 in trunk asap.

Regards,
Yannick.

summary: - Update ftp binary version
+ Update document_ftp pyftpdlib version
Revision history for this message
xrg (xrg) wrote :

FTP is deprecated alltogether, anyway. As said 3 years before, you should stop using FTP.

Revision history for this message
xrg (xrg) wrote :

... there used to be an old bug, where we had justified our decision /not/ to bother porting to the next pyftpdlib API (which was much different from 0.4, needed too much effort for an obsolete protocol).

Otherwise, your claim that packaging will suffer with such an old dependency, is valid.

Revision history for this message
YannickB (YOLO consulting) (yannick-buron) wrote :

Ok... In this case, you should really indicate in the module that document_ftp is deprecated.

Revision history for this message
xrg (xrg) wrote :

Found the old bug number: 664334 , opened at Oct. 2010 ..

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Could still be a wishlist, marking it so. Due to the amount of work required to port document_ftp to a newer version of the lib, we'd appreciate any help from the community in this area, if there's any significant interest in it.

Changed in openobject-addons:
assignee: nobody → OpenERP's Framework R&D (openerp-dev-framework)
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

Hello Oliver.

We use a lot this module, exactly as it is, it should be really cool port it, to ensure manage an standard library.

My vision:

- Port the library use, to avoid mantain a fork, it means make all with an import pyftplib and not manatain an internal fork.
- Extend the use to sftp to allow secure use.
- NOT stop use FTP because we have some legal reasons in MX at least to share trhought an FTP some invoice reports.

In terms of hours i have some budget to improve this but i dont have the correct guys to do so, for this reason, we from vauxoo can finance this porting process, to newer versions and improve functionalities to finish the end objective, have an stronger module.

Can you share your plans to know how we can contribute in terms of money and time?

Regards.

Revision history for this message
xrg (xrg) wrote : Re: [Bug 1115159] Re: Update document_ftp pyftpdlib version

On Thursday 07 February 2013, you wrote:
> - Port the library use, to avoid mantain a fork, it means make all with an
> import pyftplib and not manatain an internal fork.
Reasonable..

> - Extend the use to sftp to allow secure use.
You mean "ftps", don't you? (as in ftp with TLS) . "sftp" is rather "ftp over
ssh" which is AFAIK non-standard (and is only supported in linux clients). In
OpenERP, the huge hurdle is the way SSH server keys need to be managed inside
a python library. This makes administration of any SSH-through-openerp a pain,
and is also insecure because of the single-process nature of python.

> - NOT stop use FTP because we have some legal reasons in MX at least to
> share trhought an FTP some invoice reports.
If they mandate FTP, how come they don't also allow HTTP? You know, WebDAV is
a superset of a RESTful HTTP server...

Also, one note: the old lib was v0.4 . The new is v0.7 or something. Note that
they both are "0.x" releases, probably meaning that their API is not stable
yet.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.