dput EOFError on ftp uploads

[Jamie Strandboge]

Binary package hint: dput

When performing uploads to ubuntu or the ubuntu security archive, I get the following error message:
Traceback (most recent call last):
  File "/usr/bin/dput", line 928, in <module>
    main()
  File "/usr/bin/dput", line 877, in main
    files_to_upload, debug, ftp_mode, progress)
  File "/usr/share/dput/ftp.py", line 70, in upload
    ftp_connection.quit()
  File "/usr/lib/python2.5/ftplib.py", line 534, in quit
    resp = self.voidcmd('QUIT')
  File "/usr/lib/python2.5/ftplib.py", line 246, in voidcmd
    return self.voidresp()
  File "/usr/lib/python2.5/ftplib.py", line 221, in voidresp
    resp = self.getresp()
  File "/usr/lib/python2.5/ftplib.py", line 207, in getresp
    resp = self.getmultiline()
  File "/usr/lib/python2.5/ftplib.py", line 193, in getmultiline
    line = self.getline()
  File "/usr/lib/python2.5/ftplib.py", line 183, in getline
    if not line: raise EOFError
EOFError

Interestingly, the upload proceeds fine. This is on up to date gutsy.

$ dpkg -l|grep dput
ii dput 0.9.2.28ubuntu1

[Jamie Strandboge (jdstrand)]

This just happened to me on an up to date hardy system.

[Jamie Strandboge (jdstrand)]

Here's the hardy traceback:
...
Traceback (most recent call last):
  File "/usr/bin/dput", line 910, in <module>
    main()
  File "/usr/bin/dput", line 859, in main
    files_to_upload, debug, ftp_mode, progress=progress)
  File "/usr/share/dput/ftp.py", line 70, in upload
    ftp_connection.quit()
  File "/usr/lib/python2.5/ftplib.py", line 534, in quit
    resp = self.voidcmd('QUIT')
  File "/usr/lib/python2.5/ftplib.py", line 246, in voidcmd
    return self.voidresp()
  File "/usr/lib/python2.5/ftplib.py", line 221, in voidresp
    resp = self.getresp()
  File "/usr/lib/python2.5/ftplib.py", line 207, in getresp
    resp = self.getmultiline()
  File "/usr/lib/python2.5/ftplib.py", line 193, in getmultiline
    line = self.getline()
  File "/usr/lib/python2.5/ftplib.py", line 183, in getline
    if not line: raise EOFError
EOFError

[Chris Brotherton (protonchris)]

I am seeing this also when uploading to my ppa.

[Launchpad Janitor (janitor)]

This bug was fixed in the package dput - 0.9.2.31ubuntu1

---------------
dput (0.9.2.31ubuntu1) intrepid; urgency=low

  * Merge from debian unstable, remaining changes: (LP: #183624)
    - Exit dcut when targetting ubuntu, revu, and ppa (LP: #186275)
    - In dput.cf, set ubuntu as default_host_main and change the name of the
      ubuntu-revu stanza to revu for compatibility with previous Ubuntu
      versions of dput
    - In debian/control, set Ubuntu Core Dev as Maintainer and move Original
      maintainer to XSBC-Original-Maintainer
    - In dput, recognize 0ubuntu1 as a debian_version that requires an
      orig.tar.gz upload - Refix (LP: #17169)
  * Added stanza in dput.cf for Ubuntu PPA.

dput (0.9.2.31) unstable; urgency=low

  Welcome the new co-maintainer!

  [ Y Giridhar Appaji Nag ]
  * Maintain dput in collab-maint on alioth in git. Add Vcs-Browser and
    Vcs-Git headers in debian/control
  * Catch EOFError. In default Pure-FTPd configuration, server closes the
    connection for anonymous FTP logins (Closes: #473558)
  * Fix lintian I: hyphen-used-as-minus-sign in dput.cf and lintian W:
    manpage-has-errors-from-man in dput.1 (.P.SH changed to .SH)
  * Add Y Giridhar Appaji Nag <email address hidden> to Uploaders and add a
    DM-Upload-Allowed: yes header.
  * Allow port in FTP fqdn (defaults to 21), based on a patch submitted
    from Ubuntu by Daniel Hahler <email address hidden> (Closes: #464600)
  * Add a mentors.debian.net stanza (called mentors) to dput.cf
  * dput Suggests mini-dinstall because it is used for the local method.
    Thanks Daniel Hahler for the patch (Closes: #464596)
  * Add "-o plusdirs" to "complete" in bash_completion, so that directories
    are also used in filename completion. Thanks Daniel Hahler for the patch
    (Closes: #464592)

  [ Thomas Viehmann ]
  * Allow -u to override running the gpg signature check implied by
    --check-only. Closes: #405269
  * Switch getopt function to allow optional arguments.

 -- Sarah Hobbs <email address hidden> Sat, 03 May 2008 14:43:01 +1000

[Jamie Strandboge (jdstrand)]

I installed 0.9.2.31ubuntu1 on hardy and still see the same problems. Here is the trace back:
Traceback (most recent call last):
  File "/usr/bin/dput", line 918, in <module>
    main()
  File "/usr/bin/dput", line 867, in main
    files_to_upload, debug, ftp_mode, progress=progress, port=port)
  File "/usr/share/dput/ftp.py", line 74, in upload
    ftp_connection.quit()
  File "/usr/lib/python2.5/ftplib.py", line 534, in quit
    resp = self.voidcmd('QUIT')
  File "/usr/lib/python2.5/ftplib.py", line 246, in voidcmd
    return self.voidresp()
  File "/usr/lib/python2.5/ftplib.py", line 221, in voidresp
    resp = self.getresp()
  File "/usr/lib/python2.5/ftplib.py", line 207, in getresp
    resp = self.getmultiline()
  File "/usr/lib/python2.5/ftplib.py", line 193, in getmultiline
    line = self.getline()
  File "/usr/lib/python2.5/ftplib.py", line 183, in getline
    if not line: raise EOFError
EOFError

$ dpkg -l|grep dput
ii dput 0.9.2.31ubuntu1

[Maia Everett (linneris)]

Problem persists for me too.

[Y Giridhar Appaji Nag (appaji)]

Unlike what has been indicated in comment 4 above (Launchpad Janitor wrote on 2008-05-03) this isn't the same as #473558 in Debian.

[Miguel Ruiz (mruiz)]

It would be great to consider a fix for Hardy too. (when it appears ;-) )

Cheers!

[Launchpad Janitor (janitor)]

This bug was fixed in the package dput - 0.9.2.35ubuntu1

---------------
dput (0.9.2.35ubuntu1) jaunty; urgency=low

  * Merge from debian unstable, remaining changes: (LP: #295613)
    + dput.cf:
      - set ubuntu as default_host_main
      - set 'progress_indicator = 2'
    + dput:
      - Recognize 0ubuntu1 as a debian version that requires
        orig.tar.gz to be included in the upload.
  * sftp.py: Created dput method module to add support for uploads via sftp.
  * ftp.py:
    - Catch exceptions that are thrown on call to ftp_connection.quit()
    - Fixes LP: #183624
  * debian/control: Added suggests on bzr (as we use the bzrlib.transport)
  * debian/rules: Updated to install sftp.py
  * dput:
    - Added ability to pass an argument to the host configuration by placing
      a colon after the hostname. The variable with the same name of the host
      will be replaced with the string following the colon. (LP: #196797)
    - Properly check to see if config stanza exists for host. (LP: #196786)
    - Updated version string to reflect current version.
  * dput.cf: Updated ppa stanza to make use of argument support.
  * dput.cf.5: Updated man page to note support for sftp transport and host args
  * dput.1: Updated man page to document host argument feature and sftp support.

dput (0.9.2.35) unstable; urgency=low

  * dput.cf: add ssh-upload pointing to ssh.upload.debian.org.
    Closes: #499731.

 -- <email address hidden> (Cody A.W. Somerville) Sat, 08 Nov 2008 10:51:36 -0500