Merge lp:~zaber/openobject-addons/document-ftp-ports-6.1 into lp:openobject-addons/6.1

Proposed by Don Kirkby
Status: Needs review
Proposed branch: lp:~zaber/openobject-addons/document-ftp-ports-6.1
Merge into: lp:openobject-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
document_ftp/ftpserver/__init__.py (+1/-1)
To merge this branch: bzr merge lp:~zaber/openobject-addons/document-ftp-ports-6.1
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+115825@code.launchpad.net

Description of the change

Treat the FTP passive ports configuration setting as a range, not two separate ports.

To post a comment you must log in.

Unmerged revisions

6897. By Don Kirkby

[FIX] Treat FTP server passive port as a range.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'document_ftp/ftpserver/__init__.py'
2--- document_ftp/ftpserver/__init__.py 2011-10-11 20:05:49 +0000
3+++ document_ftp/ftpserver/__init__.py 2012-07-19 20:12:19 +0000
4@@ -32,7 +32,7 @@
5 PASSIVE_PORTS = None
6 pps = config.get('ftp_server_passive_ports', '').split(':')
7 if len(pps) == 2:
8- PASSIVE_PORTS = int(pps[0]), int(pps[1])
9+ PASSIVE_PORTS = range(int(pps[0]), int(pps[1]) + 1)
10
11 class ftp_server(threading.Thread):
12 def log(self, level, message):