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.