Skip to content

Conversation

@michurin
Copy link

@michurin michurin commented Mar 16, 2025

Hi!

I'm not sure this project is supported, and I didn't check other pull requests. However, I would like to share this update.

Changes:

I would like to keep playing with this bunch of code. Improve error handling, write tests... Would you vote for this?

upd

  • added one golden flow integration test
  • added CI github actions (looks like this) for
    • build on different versions of Go
    • linting (config tuned)
    • testing

upd2

I enriched logging and error. You can see changes here michurin#2. However, it looks like subject of next poll request.

upd3

Dependencies has to be updated again :-) michurin#5

upd4

One more nice to be stolen thing michurin#7. Prevention of 100% CPU consumption on server side in case of some errors: context canceled, port already occupied, lack of process permissions...

upd5

Migrate github.com/urfave/cli/v2 -> v3, according suggestion #179michurin#10

upd6

By the way, it could be nice to use session context in session handler — https://github.com/michurin/quicssh/pull/11/files#diff-366e46a40f6f60b4f7614eb0976bb51820364bf5ca6ccc4787eb49d7bdbef3e6R69

upd7

Added --idletimeout option. Useful for coupling quicssh with netpunching. Also added behavioral test. michurin#8

@michurin michurin mentioned this pull request Mar 29, 2025
@michurin michurin closed this Apr 8, 2025
@michurin michurin deleted the dev branch April 8, 2025 08:28
@michurin michurin restored the dev branch April 9, 2025 03:47
@michurin michurin reopened this Apr 9, 2025
@socket-security
Copy link

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
golang/github.com/quic-go/[email protected]v0.50.0 None 0 3.16 MB
golang/github.com/urfave/cli/[email protected]v2.27.6 None 0 6.73 MB
golang/golang.org/x/[email protected]v0.37.0 None 0 6.63 MB

View full report↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

can I set the timeout ?

1 participant