Merge lp:~adiroiban/twisted/twisted-5046 into lp:twisted

Proposed by Adi Roiban
Status: Merged
Merge reported by: Jean-Paul Calderone
Merged at revision: not available
Proposed branch: lp:~adiroiban/twisted/twisted-5046
Merge into: lp:twisted
Diff against target: 11 lines (+0/-1)
1 file modified
twisted/protocols/ftp.py (+0/-1)
To merge this branch: bzr merge lp:~adiroiban/twisted/twisted-5046
Reviewer Review Type Date Requested Status
Twisted-dev Pending
Review via email: mp+74920@code.launchpad.net

Description of the change

I would like to see ticket 5046 closed, so I started a branch to fix the issues raised by that ticket.

At a first look at the ftp code, it looks like only 'userotp' should be removed from ftp.py.

Please let me know if you think there should be other changes and I will be happy to investigate them.

Regards,
Adi

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'twisted/protocols/ftp.py'
--- twisted/protocols/ftp.py 2011-04-10 14:17:53 +0000
+++ twisted/protocols/ftp.py 2011-09-11 19:32:32 +0000
@@ -135,7 +135,6 @@
135 MKD_REPLY: '257 "%s" created',135 MKD_REPLY: '257 "%s" created',
136136
137 # -- 300's --137 # -- 300's --
138 'userotp': '331 Response to %s.', # ???
139 USR_NAME_OK_NEED_PASS: '331 Password required for %s.',138 USR_NAME_OK_NEED_PASS: '331 Password required for %s.',
140 GUEST_NAME_OK_NEED_EMAIL: '331 Guest login ok, type your email address as password.',139 GUEST_NAME_OK_NEED_EMAIL: '331 Guest login ok, type your email address as password.',
141 NEED_ACCT_FOR_LOGIN: '332 Need account for login.',140 NEED_ACCT_FOR_LOGIN: '332 Need account for login.',

Subscribers

People subscribed via source and target branches

to all changes: