Hello! I just stumbled over the fact that the ControlMaster process does not do "cd /" but remains where the ssh client was invoked. Is this intentional, or should i open a bug report on that? (Also i still have not found the time, would a patch to ssh-agent that creates a signal handler and "somehow" does the equivalent to "ssh-add -D" be interesting? For now i iterate over temporary directory entries in order to be able to do 'SSH_AUTH_SOCK="$a" ssh-add -D' upon LID close, nicer, especially in conjunction with private temporary directories, would be "pkill -USR1 ssh-agent", for example.) Thanks, and Ciao! from Germany, --steffen | |Der Kragenbaer, The moon bear, |der holt sich munter he cheerfully and one by one |einen nach dem anderen runter wa.ks himself off |(By Robert Gernhardt)
On Wed, Mar 31, 2021 at 12:33:43AM +0200, Steffen Nurpmeso <steffen at sdaoden.eu> wrote:> Hello! > > I just stumbled over the fact that the ControlMaster process does > not do "cd /" but remains where the ssh client was invoked. Is > this intentional, or should i open a bug report on that? > > [snip] > > Thanks, and Ciao! from Germany, > > --steffenHi, I reported this too a little while ago and there was no interest. But it is a pain when your cwd is a USB stick that you want to umount. I've created a pull request that fixes it: https://github.com/openssh/openssh-portable/pull/239 But it's failing checks on 4 Ubuntu kitchensink/sk versions. This page shows the error message: https://github.com/openssh/openssh-portable/actions/runs/704178803 No files were found with the provided path: regress/*.log regress/valgrind-out/. No artifacts will be uploaded. Does anyone know if the tests can/should be altered to cope with chdir / having been introduced? cheers, raf
Hello! raf wrote in <20210331075157.sgfbkvofdx6dgiyi at raf.org>: |On Wed, Mar 31, 2021 at 12:33:43AM +0200, Steffen Nurpmeso <steffen at sdao\ |den.eu> wrote: |> I just stumbled over the fact that the ControlMaster process does |> not do "cd /" but remains where the ssh client was invoked. Is |> this intentional, or should i open a bug report on that? |> |> [snip] ... |I reported this too a little while ago and there was no |interest. But it is a pain when your cwd is a USB stick |that you want to umount. I've created a pull request Yeah, and encrypted volumes (in my case). |that fixes it: | | https://github.com/openssh/openssh-portable/pull/239 That is great! I had not looked at github. (That pull i could not have found by then?) |But it's failing checks on 4 Ubuntu kitchensink/sk |versions. | |This page shows the error message: | | https://github.com/openssh/openssh-portable/actions/runs/704178803 | | No files were found with the provided path: | regress/*.log regress/valgrind-out/. No artifacts | will be uploaded. I am not at github so cannot see the log, but since configure failed also locally, a "autoreconf" seems to fix it. After that: make[1]: Leaving directory '/tmp/x/openssh.tar_bomb_git/regress' unit tests passed echo all tests passed all tests passed (8.5p1 plus your patch.) It would be nice if that patch made it in. Thanks, and Ciao from Germany, --steffen | |Der Kragenbaer, The moon bear, |der holt sich munter he cheerfully and one by one |einen nach dem anderen runter wa.ks himself off |(By Robert Gernhardt)