Bug #33378 “Sftp can't copy directories” : Bugs : openssh package : Ubuntu

Revision history for this message

In OpenSSH Portable Bugzilla #520, Mouring (mouring) wrote on 2003-03-27:

#1

fts would be easy enough to use for recursive. I think 'put' can be written in
a matter of a night or two (Assuming one is awake <chuckle>), but there is one
minor downfall to fts. It assuming all access is local. And unlike glob it
does not support the ability to override opendir(), readdir(), closedir(), lstat
() and stat().

Unless there is something I'm missing get and put can't both use fts effectly
without forking the code and rewriting small chunks of it.

Would be nice if fts could be modified to support virtualizing those function
calls, but with it having a chance of going into POSIX (some century) I doubt
the API can change much.

Revision history for this message

In OpenSSH Portable Bugzilla #520, Cmp-uiuc (cmp-uiuc) wrote on 2004-12-07:

#2

Created an attachment (id=749)
Add recurse mode to sftp put command via fts.

Add recurse mode to sftp put command via fts.

Initial work was done against openssh snapshots from 2004/01. The patch is
attached here as a contribution to anyone considering adding this feature to
sftp. I assume it would require some reworking to be applied against a modern
OpenSSH snapshot.

Revision history for this message

In OpenSSH Portable Bugzilla #520, Cmp-uiuc (cmp-uiuc) wrote on 2004-12-07:

#3

(From update of attachment 749)
This patch was derived in part from Jared Yanovich's sftp patch posted to
openssh-unix-dev in Oct 2003.

http://marc.theaimsgroup.com/?l=openssh-unix-dev&m=106498593122963&w=2

I removed the code in my version of the patch that handled the 'recurse'
command in sftp but forgot to remove the documentation about that command.
Trivial to either add back or to remove the references to it in the docs.

Revision history for this message

Tero Karvinen (karvinen+launchpad) wrote on 2006-03-02:

#4

Sftp can't copy directories, even though the protocol supports it.

$ sftp tkarvine@127.0.0.69
Connecting to 127.0.0.69...
sftp> ls
somedir
sftp> get somedir
Fetching /home/tee/3004WTMi to 3004WTMi
Cannot download non-regular file: /home/tkarvine/somedir

Currently, every method of using ssh to transfer files requires a manual to use: nautilus ssh requires user to know home dir (https://launchpad.net/distros/ubuntu/+source/nautilus/+bug/30039), scp has tricky syntax and terse error messages, sftp can't transfer directories.

Revision history for this message

Tero Karvinen (karvinen+launchpad) wrote on 2006-03-02:

#5

Minor typo in example, should be:
Fetching /home/tkarvine/somedir to somedir

Revision history for this message

Loye Young (loyeyoung) wrote on 2008-02-08:

#6

>Currently, every method of using ssh to transfer files requires a manual to use

This may have been fixed already. sshfs mounts the remote directory on the local file system, enabling transparent transfer of files. I believe that it invokes sftp to accomplish the task.

Revision history for this message

Scott Kitterman (kitterman) wrote on 2008-02-08:

#7

This definitely works. I use it almost every day.

Changed in openssh:
status:	New → Fix Released

Revision history for this message

Colin Watson (cjwatson) wrote on 2008-02-11:

#8

sshfs is great, but doesn't justify closing this bug IMO; sftp could support this too and it would be useful for it to do so. In fact, it's an open bug upstream as well.

Changed in openssh:
status:	Fix Released → Confirmed

Bug Watch Updater (bug-watch-updater) on 2008-02-11

Changed in openssh:
status:	Unknown → Confirmed

Revision history for this message

Tero Karvinen (karvinen+launchpad) wrote on 2008-02-24:

#9

Also, normal user is not allowed to use sshfs by default. It is not obvious how to securely allow this, and it doesn't "just work". Sshfs is pretty new and very reliable yet.

Revision history for this message

komputes (komputes) wrote on 2009-03-21:

#10

Confirming this is still an issue on Intrepid with openssh-client 1:5.1p1-3ubuntu1. sshfs has its own issues.

Revision history for this message

Scott Ritchie (scottritchie) wrote on 2009-09-03:

#11

As a human, I would expect sftp to be more or less syntax and feature equivalent to command line ftp. Before I took the time to figure out scp I would use the ridiculous workaround of tarring an entire folder before copying it with sftp.

Revision history for this message

In OpenSSH Portable Bugzilla #520, Scott Ritchie (scottritchie) wrote on 2009-09-06:

#12

So this bug has been stale for four years, but this feature is still wanted. See, for instance: https://bugs.launchpad.net/ubuntu/+source/openssh/+bug/33378

Has the code here bitrotted? Would someone mind taking care of this wonderful feature? You'll have my gratitude :)

Revision history for this message

In OpenSSH Portable Bugzilla #520, Damien Miller (djm) wrote on 2009-12-08:

#13

We now have get and put recursion in CVS, developed by Carlos Silva part of the Google Summer of Code. This will be in the openssh-5.4

Bug Watch Updater (bug-watch-updater) on 2009-12-08

Changed in openssh:
status:	Confirmed → Fix Released

Revision history for this message

In OpenSSH Portable Bugzilla #520, Damien Miller (djm) wrote on 2010-04-16:

#14

Mass move of bugs RESOLVED->CLOSED following the release of openssh-5.5p1

Revision history for this message

Launchpad Janitor (janitor) wrote on 2010-05-09:

#15

Download full text (6.4 KiB)

This bug was fixed in the package openssh - 1:5.5p1-3ubuntu1

---------------
openssh (1:5.5p1-3ubuntu1) maverick; urgency=low

  * Resynchronise with Debian. Remaining changes:
    - Add support for registering ConsoleKit sessions on login.
    - Drop openssh-blacklist and openssh-blacklist-extra to Suggests; they
      take up a lot of CD space, and I suspect that rolling them out in
      security updates has covered most affected systems now.
    - Convert to Upstart. The init script is still here for the benefit of
      people running sshd in chroots.
    - Install apport hook.
  * Stop setting OOM adjustment in Upstart job; sshd does it itself now.

openssh (1:5.5p1-3) unstable; urgency=low

  * Discard error messages while checking whether rsh, rlogin, and rcp
    alternatives exist (closes: #579285).
  * Drop IDEA key check; I don't think it works properly any more due to
    textual changes in error output, it's only relevant for direct upgrades
    from truly ancient versions, and it breaks upgrades if
    /etc/ssh/ssh_host_key can't be loaded (closes: #579570).

openssh (1:5.5p1-2) unstable; urgency=low

* Use dh_installinit -n, since our maintainer scripts already handle this
more carefully (thanks, Julien Cristau).

openssh (1:5.5p1-1) unstable; urgency=low

  * New upstream release:
    - Unbreak sshd_config's AuthorizedKeysFile option for $HOME-relative
      paths.
    - Include a language tag when sending a protocol 2 disconnection
      message.
    - Make logging of certificates used for user authentication more clear
      and consistent between CAs specified using TrustedUserCAKeys and
      authorized_keys.

openssh (1:5.4p1-2) unstable; urgency=low

  * Borrow patch from Fedora to add DNSSEC support: if glibc 2.11 is
    installed, the host key is published in an SSHFP RR secured with DNSSEC,
    and VerifyHostKeyDNS=yes, then ssh will no longer prompt for host key
    verification (closes: #572049).
  * Convert to dh(1), and use dh_installdocs --link-doc.
  * Drop lpia support, since Ubuntu no longer supports this architecture.
  * Use dh_install more effectively.
  * Add a NEWS.Debian entry about changes in smartcard support relative to
    previous unofficial builds (closes: #231472).

openssh (1:5.4p1-1) unstable; urgency=low

  * New upstream release (LP: #535029).
    - After a transition period of about 10 years, this release disables SSH
      protocol 1 by default. Clients and servers that need to use the
      legacy protocol must explicitly enable it in ssh_config / sshd_config
      or on the command-line.
    - Remove the libsectok/OpenSC-based smartcard code and add support for
      PKCS#11 tokens. This support is enabled by default in the Debian
      packaging, since it now doesn't involve additional library
      dependencies (closes: #231472, LP: #16918).
    - Add support for certificate authentication of users and hosts using a
      new, minimal OpenSSH certificate format (closes: #482806).
    - Added a 'netcat mode' to ssh(1): "ssh -W host:port ...".
    - Add the ability to revoke keys in sshd(8) and ssh(1). (For the Debian
      package, this overlaps with the key blacklisting facil...

This bug was fixed in the package openssh - 1:5.5p1-3ubuntu1

---------------
openssh (1:5.5p1-3ubuntu1) maverick; urgency=low

* Resynchronise with Debian.  Remaining changes:
    - Add support for registering ConsoleKit sessions on login.
    - Drop openssh-blacklist and openssh-blacklist-extra to Suggests; they
      take up a lot of CD space, and I suspect that rolling them out in
      security updates has covered most affected systems now.
    - Convert to Upstart.  The init script is still here for the benefit of
      people running sshd in chroots.
    - Install apport hook.
  * Stop setting OOM adjustment in Upstart job; sshd does it itself now.

openssh (1:5.5p1-3) unstable; urgency=low

* Discard error messages while checking whether rsh, rlogin, and rcp
    alternatives exist (closes: #579285).
  * Drop IDEA key check; I don't think it works properly any more due to
    textual changes in error output, it's only relevant for direct upgrades
    from truly ancient versions, and it breaks upgrades if
    /etc/ssh/ssh_host_key can't be loaded (closes: #579570).

openssh (1:5.5p1-2) unstable; urgency=low

* Use dh_installinit -n, since our maintainer scripts already handle this
    more carefully (thanks, Julien Cristau).

openssh (1:5.5p1-1) unstable; urgency=low

* New upstream release:
    - Unbreak sshd_config's AuthorizedKeysFile option for $HOME-relative
      paths.
    - Include a language tag when sending a protocol 2 disconnection
      message.
    - Make logging of certificates used for user authentication more clear
      and consistent between CAs specified using TrustedUserCAKeys and
      authorized_keys.

openssh (1:5.4p1-2) unstable; urgency=low

* Borrow patch from Fedora to add DNSSEC support: if glibc 2.11 is
    installed, the host key is published in an SSHFP RR secured with DNSSEC,
    and VerifyHostKeyDNS=yes, then ssh will no longer prompt for host key
    verification (closes: #572049).
  * Convert to dh(1), and use dh_installdocs --link-doc.
  * Drop lpia support, since Ubuntu no longer supports this architecture.
  * Use dh_install more effectively.
  * Add a NEWS.Debian entry about changes in smartcard support relative to
    previous unofficial builds (closes: #231472).

openssh (1:5.4p1-1) unstable; urgency=low

* New upstream release (LP: #535029).
    - After a transition period of about 10 years, this release disables SSH
      protocol 1 by default.  Clients and servers that need to use the
      legacy protocol must explicitly enable it in ssh_config / sshd_config
      or on the command-line.
    - Remove the libsectok/OpenSC-based smartcard code and add support for
      PKCS#11 tokens.  This support is enabled by default in the Debian
      packaging, since it now doesn't involve additional library
      dependencies (closes: #231472, LP: #16918).
    - Add support for certificate authentication of users and hosts using a
      new, minimal OpenSSH certificate format (closes: #482806).
    - Added a 'netcat mode' to ssh(1): "ssh -W host:port ...".
    - Add the ability to revoke keys in sshd(8) and ssh(1).  (For the Debian
      package, this overlaps with the key blacklisting facility added in
      openssh 1:4.7p1-9, but with different file formats and slightly
      different scopes; for the moment, I've roughly merged the two.)
    - Various multiplexing improvements, including support for requesting
      port-forwardings via the multiplex protocol (closes: #360151).
    - Allow setting an explicit umask on the sftp-server(8) commandline to
      override whatever default the user has (closes: #496843).
    - Many sftp client improvements, including tab-completion, more options,
      and recursive transfer support for get/put (LP: #33378).  The old
      mget/mput commands never worked properly and have been removed
      (closes: #270399, #428082).
    - Do not prompt for a passphrase if we fail to open a keyfile, and log
      the reason why the open failed to debug (closes: #431538).
    - Prevent sftp from crashing when given a "-" without a command.  Also,
      allow whitespace to follow a "-" (closes: #531561).

* Fix 'debian/rules quilt-setup' to avoid writing .orig files if some
    patches apply with offsets.
  * Include debian/ssh-askpass-gnome.png in the Debian tarball now that
    we're using a source format that permits this, rather than messing
    around with uudecode.
  * Drop compatibility with the old gssapi mechanism used in ssh-krb5 <<
    3.8.1p1-1.  Simon Wilkinson refused this patch since the old gssapi
    mechanism was removed due to a serious security hole, and since these
    versions of ssh-krb5 are no longer security-supported by Debian I don't
    think there's any point keeping client compatibility for them.
  * Fix substitution of ETC_PAM_D_SSH, following the rename in 1:4.7p1-4.
  * Hardcode the location of xauth to /usr/bin/xauth rather than
    /usr/bin/X11/xauth (thanks, Aron Griffis; closes: #575725, LP: #8440).
    xauth no longer depends on x11-common, so we're no longer guaranteed to
    have the /usr/bin/X11 symlink available.  I was taking advantage of the
    /usr/bin/X11 symlink to smooth X's move to /usr/bin, but this is far
    enough in the past now that it's probably safe to just use /usr/bin.
  * Remove SSHD_OOM_ADJUST configuration.  sshd now unconditionally makes
    itself non-OOM-killable, and doesn't require configuration to avoid log
    spam in virtualisation containers (closes: #555625).
  * Drop Debian-specific removal of OpenSSL version check.  Upstream ignores
    the two patchlevel nybbles now, which is sufficient to address the
    original reason this change was introduced, and it appears that any
    change in the major/minor/fix nybbles would involve a new libssl package
    name.  (We'd still lose if the status nybble were ever changed, but that
    would mean somebody had packaged a development/beta version rather than
    a proper release, which doesn't appear to be normal practice.)
  * Drop most of our "LogLevel SILENT" (-qq) patch.  This was originally
    introduced to match the behaviour of non-free SSH, in which -q does not
    suppress fatal errors, but matching the behaviour of OpenSSH upstream is
    much more important nowadays.  We no longer document that -q does not
    suppress fatal errors (closes: #280609).  Migrate "LogLevel SILENT" to
    "LogLevel QUIET" in sshd_config on upgrade.
  * Policy version 3.8.4:
    - Add a Homepage field.
 -- Colin Watson <cjwatson@ubuntu.com>   Sun, 09 May 2010 18:04:01 +0200

Changed in openssh (Ubuntu):
status:	Confirmed → Fix Released

Affects		Status	Importance	Assigned to	Milestone
	portable OpenSSH	Fix Released	Unknown	openssh-portable-bugs #520
	openssh (Ubuntu)	Fix Released	Wishlist	Unassigned

Ubuntu
openssh package

Sftp can't copy directories

Bug Description

Related branches

Other bug subscribers

Remote bug watches

Ubuntuopenssh package

Sftp can't copy directories

Bug Description

Related branches

Other bug subscribers

Remote bug watches

Ubuntu
openssh package