Skip to content

Extend session with active SSH connection #27

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Extend session with active SSH connection #27

wants to merge 5 commits into from

Conversation

@rcrowe
Copy link

@rcrowe rcrowe commented Oct 26, 2025 •
edited
Loading

  1. The Breakpoint session does not automatically extend itself if an SSH connection is active. You need to explicitly extend the session with breakpoint extend.

✅ "Active" here is the connection has bytes written to it
✅ Set duration_auto_extend to enable the functionality; the value is a duration string, like duration, that is added to the duration timeout.

  • Callback when write takes place, to extend breakpoint session
  • Configuration: Allow opt-in to behaviour & how long session is extended
  • Replace mgr.ExtendWait with non-announcing method

rcrowe added 2 commits October 26, 2025 18:10
@rcrowe
trigger notification when session write happens
5a7172b 
When a write takes place on the session, if configured, a callback
function is triggered.

Callback is only triggered at most, once every 1 minute. It's designed
to be best effort and not be accurate, so as not to slow the write path.
@rcrowe
trigger session extension on ssh session write
d0b15ac 
POC for now, just to show it working.
rcrowe
rcrowe commented Oct 26, 2025
View reviewed changes
rcrowe added 3 commits October 27, 2025 06:19
@rcrowe
Conditionally announce
4da1f65
@rcrowe
enable & configure auto extension time
b9c031a 
set `duration_auto_extend` to a value such as `1m` to enable & extend a
session by this amount of time.

Minimum 1min extension
@rcrowe
Trigger more often
d80b56a 
In combination of the minimum auto extend time, we can trigger
notification more often
@rcrowe rcrowe marked this pull request as ready for review October 27, 2025 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@annervisser annervisser Awaiting requested review from annervisser

@hugosantos hugosantos Awaiting requested review from hugosantos

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rcrowe