Volker Diels-Grabsch
2021-Jun-03 16:31 UTC
Reminder: Add ssh_config equivalents of -N, -n and -f
Ping. [ If there's anything I can do to reduce the barrier, such as posting the pull request as patch files to the mailing list, or anything, please let me know. ] Best regards, Volker Damien Miller wrote:> On Tue, 4 May 2021, Volker Diels-Grabsch wrote: > > > Dear Damien, > > > > Damien Miller wrote: > > > On Fri, 26 Feb 2021, Volker Diels-Grabsch wrote: > > > > > > > > Is it still possible to include the ssh_config improvements which I > > > > proposed last Sunday on this mailing list? > > > > > > No sorry, but I'll look at the right after release. We'll be doing > > > another release in 2-3 months, so not long to wait. > > > > Meanwhile, that version (8.6) was released as well. > > > > Is there any chance to include this handy patch in the release thereafter (8.7)? > > > > For your convenience: https://github.com/openssh/openssh-portable/pull/231 > > Sorry - have been very busy with the day job but will start catching up on > patch requests soon. > > -d > _______________________________________________ > openssh-unix-dev mailing list > openssh-unix-dev at mindrot.org > https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev-- Volker Diels-Grabsch ----<<<((()))>>>----
On Thu, 3 Jun 2021, Volker Diels-Grabsch wrote:> Ping. > > [ If there's anything I can do to reduce the barrier, such as posting > the pull request as patch files to the mailing list, or anything, > please let me know. ]Looking at these now. I'm not sure about the naming of NoShell, because it's really about not starting any kind of remote session. E.g. the existing -N talks about not executing a remote command. Talking about shells I think runs the risk that people will assume it somehow skips the remote shell for command execution. They would probably find out soon enough that they were wrong, but IMO it's better to avoid it altogether. So the options are to rename the option to something like NoSession, find a way to reuse the existing RemoteCommand option (though that is IMO a little ugly) or perhaps introduce a new "SessionType" option that can subsume functionality of both the -N (SessionType=none) and -s (SessionType=subsystem) flags. What do you think? -d