bugzilla-daemon at bugzilla.mindrot.org
2020-Mar-12 23:21 UTC
[Bug 3135] New: SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Bug ID: 3135
Summary: SFTP Assumes quiet in batch mode - This regression
breaks integration
Product: Portable OpenSSH
Version: 8.2p1
Hardware: Other
OS: Linux
Status: NEW
Severity: normal
Priority: P5
Component: sftp
Assignee: unassigned-bugs at mindrot.org
Reporter: matt at roystontech.com.au
Prior to 2013, sftp in batch mode (-b) produced output that included
what was done. Quiet mode could be specified as well if desired.
Batch mode now assumes quiet mode and cannot be overridden.
This breaks integration which expects sftp output but uses batch mode.
Quiet as the default might be ok, but there is no way to 'unquiet'
I will be providing a pull on github that checks if quiet is set - if
it's not set, assume quiet, if it's set to no, honour that setting.
This provides the default functionality which is now current, but
provides the ability to override.
--
You are receiving this mail because:
You are watching the assignee of the bug.
bugzilla-daemon at bugzilla.mindrot.org
2020-Mar-13 00:01 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |djm at mindrot.org
--- Comment #1 from Damien Miller <djm at mindrot.org> ---
I don't understand what you're referring to. sftp does produce output
by default in batch mode:
$ cat > /tmp/b
cd /bin
get ls /tmp
get xxls /tmp
$ sftp -b /tmp/b localhost
sftp> cd /bin
sftp> get ls /tmp
sftp> get xxls /tmp
File "/bin/xxls" not found.
--
You are receiving this mail because:
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.
bugzilla-daemon at bugzilla.mindrot.org
2020-Mar-13 00:31 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135 --- Comment #2 from Matthew Parslow <matt at roystontech.com.au> --- It does have some output, but it's in quiet mode. https://github.com/openssh/openssh-portable/commit/9303e6527bb5ca7630c765f28624702c212bfd6c?fbclid=IwAR3KCDxObbHrEBsHNUrHDxFfv4FT6wULg2udwPq6ZhPZvHkbwYdA2qbwnYI#diff-bfa9ee6de996100a65619653d189809fR2195 shows where it sets quiet mode. example: $ ls -lrt total 0 $ cat /tmp/test/commands ls /tmp/test/ get /tmp/test/* $ sftp -b /tmp/test/commands testhost sftp> ls /tmp/test /tmp/test/1file /tmp/test/2dir /tmp/test/3file sftp> get /tmp/test/* Cannot download non-regular file: /tmp/test/2dir/ $ ls 1file 3file Compare to: matt at ludev:/tmp/dest$ sftp testhost < /tmp/test/commands Connected to testhost. sftp> ls /tmp/test/ /tmp/test/1file /tmp/test/2dir /tmp/test/3file sftp> get /tmp/test/* Fetching /tmp/test/1file to 1file Fetching /tmp/test/2dir/ to 2dir Cannot download non-regular file: /tmp/test/2dir/ Fetching /tmp/test/3file to 3file -- You are receiving this mail because: You are watching someone on the CC list of the bug. You are watching the assignee of the bug.
bugzilla-daemon at bugzilla.mindrot.org
2020-Mar-13 02:44 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
Assignee|unassigned-bugs at mindrot.org |djm at mindrot.org
Attachment #3363| |ok?
Flags| |
--- Comment #3 from Damien Miller <djm at mindrot.org> ---
Created attachment 3363
--> https://bugzilla.mindrot.org/attachment.cgi?id=3363&action=edit
Add -N flag to override quiet mode
Ah, I changed that because a few people thought the verbose output was
too noisy for scripted use.
Anyway, this patch adds a -N flag ("noisy") that clears quiet mode:
[djm at djm openssh]$ ./sftp -Nb /tmp/b localhost
Connected to fuyu.
sftp> cd /bin
sftp> get ls /tmp
Fetching /bin/ls to /tmp/ls
sftp> get xxls /tmp
File "/bin/xxls" not found.
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
bugzilla-daemon at mindrot.org
2020-Mar-13 02:49 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |dtucker at dtucker.net
Attachment #3363|ok? |ok?(dtucker at dtucker.net)
Flags| |
Blocks| |3117
Attachment #3363|ok?(dtucker at dtucker.net) |ok?
Flags| |
Referenced Bugs:
https://bugzilla.mindrot.org/show_bug.cgi?id=3117
[Bug 3117] Tracking bug for 8.3 release
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
bugzilla-daemon at mindrot.org
2020-Mar-13 02:49 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #3363|ok? |ok?(dtucker at dtucker.net)
Flags| |
--
You are receiving this mail because:
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.
bugzilla-daemon at mindrot.org
2020-Mar-13 02:51 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #3363|ok?(dtucker at dtucker.net) |
Flags| |
Attachment #3363|0 |1
is obsolete| |
Attachment #3364| |ok?(dtucker at dtucker.net)
Flags| |
--- Comment #4 from Damien Miller <djm at mindrot.org> ---
Created attachment 3364
--> https://bugzilla.mindrot.org/attachment.cgi?id=3364&action=edit
correct patch
oops, wrong patch. This one does it correctly.
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
bugzilla-daemon at mindrot.org
2020-Apr-03 04:47 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #5 from Damien Miller <djm at mindrot.org> ---
this has been committed and will be in openssh-8.3 - thanks
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
bugzilla-daemon at mindrot.org
2021-Mar-03 22:54 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |CLOSED
--- Comment #6 from Damien Miller <djm at mindrot.org> ---
close bugs that were resolved in OpenSSH 8.5 release cycle
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.
bugzilla-daemon at mindrot.org
2023-Jan-13 02:42 UTC
[Bug 3135] SFTP Assumes quiet in batch mode - This regression breaks integration
https://bugzilla.mindrot.org/show_bug.cgi?id=3135
Damien Miller <djm at mindrot.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #3364|ok?(dtucker at dtucker.net) |
Flags| |
--
You are receiving this mail because:
You are watching someone on the CC list of the bug.
You are watching the assignee of the bug.