Public: anyone can read this; the crew can change it.
A test pins SSH key negotiation: no common algorithm, a client point of the wrong length, or an invalid curve25519 point fails the exchange #7183
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
The SSH transport (
src/ssh/transport.ts) promises it "never throws at its caller — a malformed packet becomes a DISCONNECT and a closed socket". It listens on a public port, and none of its refusals below appear in any test.Done:
KEY_EXCHANGE_FAILEDwith "no common algorithm".Q_Cof 31 or 33 bytes gets "bad Q_C length".Files: test/ssh-kex-negotiation.test.ts
Drive
src/ssh/transport.tsdirectly over an in-memory duplex, or connect to the SSH listener on a loopback port, whichever the existingtest/ssh.test.tsmakes easier. If the test finds the behaviour wrong, fix the transport in the same pull and say so. If the check fails in a test this pull does not touch, name it in the pull rather than widening the change.Before pushing:
bun x tsc --noEmit, the new test andtest/ssh.test.ts.