tg123/sshpiper

Error when chaining plugins

Closed this issue · 6 comments

pchampio commented 
sudo ./out/sshpiperd ./out/totp --root ./workingdir/ -- ./out/workingdir --root ./workingdir/
INFO[0000] starting sshpiperd version: (devel), 318ac45ce, 2023-10-13T09:02:23Z, go1.21.0
INFO[0000] found host keys [/etc/ssh/ssh_host_ed25519_key]
INFO[0000] loading host key /etc/ssh/ssh_host_ed25519_key
INFO[0000] starting child process plugin: [./out/totp --root ./workingdir/]
INFO[0000] starting child process plugin: [./out/workingdir --root ./workingdir/]
ERRO[0000] plugin ./out/workingdir recv logs error: rpc error: code = Unavailable desc = connection error: desc = "error reading server preface: http2: frame too large"
FATA[0000] rpc error: code = Unavailable desc = connection error: desc = "error reading server preface: http2: frame too large"

Without the chain -- it works fine but I can only enable one plugin

pchampio commented

After reboot it single plugin does not work anymore:

sudo ./out/sshpiperd ./out/workingdir --root ./workingdir/
INFO[0000] starting sshpiperd version: (devel), 318ac45ce, 2023-10-13T09:02:23Z, go1.21.0
INFO[0000] found host keys [/etc/ssh/ssh_host_ed25519_key]
INFO[0000] loading host key /etc/ssh/ssh_host_ed25519_key
INFO[0000] starting child process plugin: [./out/workingdir --root ./workingdir/]
FATA[0000] rpc error: code = Unavailable desc = connection error: desc = "error reading server preface: http2: frame too large"
pchampio commented

More info, when i compile sshpiperd it does not work, but when I use the one in the release it works.
Might be related to #242 ??

tg123 commented

let me double check
it was technically covered by e2e test

tg123 commented

cant repro
any hint?

pchampio commented

Might be a fmt.Printf getting used in my plugin rather than log.Printf("%v", conn)

tg123 commented

ah yup, stdin and stdou are used for communication
you may consider stderr

  • 👍1