Skip to content
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.

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

Timeout Notification Signal #816

Closed
totaam opened this issue Feb 24, 2015 · 9 comments
Closed

Timeout Notification Signal #816

totaam opened this issue Feb 24, 2015 · 9 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Feb 24, 2015

Issue migrated from trac ticket # 816

component: server | priority: major | resolution: fixed

2015-02-24 22:55:31: afarr created the issue


With idle-timeouts, would like to receive a signal x seconds before timeout occurs. Perhaps as a separate parameter (--idle-timeout-warning=x)?

@totaam
Copy link
Collaborator Author

totaam commented Feb 25, 2015

2015-02-25 01:07:19: antoine changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Feb 25, 2015

2015-02-25 01:07:19: antoine commented


Follow up from #803.

@totaam
Copy link
Collaborator Author

totaam commented Feb 25, 2015

2015-02-25 10:37:41: antoine changed status from assigned to new

@totaam
Copy link
Collaborator Author

totaam commented Feb 25, 2015

2015-02-25 10:37:41: antoine changed owner from antoine to sbennett

@totaam
Copy link
Collaborator Author

totaam commented Feb 25, 2015

2015-02-25 10:37:41: antoine commented


  • r8703 adds the grace timeout which can be overriden by hooking the idle_grace_timeout_cb function, by default the client will get a notification message at 90% of the time. The percentage can be changed using the env var XPRA_GRACE_PERCENT.
  • r8705 makes us slow down screen updates to just 2 per second when we reach the grace period (the code is reusable)

Ready for testing!

@totaam
Copy link
Collaborator Author

totaam commented Apr 22, 2015

2015-04-22 05:03:18: antoine commented


@sbennett: bump!

@totaam
Copy link
Collaborator Author

totaam commented May 8, 2015

2015-05-08 09:52:24: antoine changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented May 8, 2015

2015-05-08 09:52:24: antoine set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented May 8, 2015

2015-05-08 09:52:24: antoine commented


I've been told this works as expected.

@totaam totaam closed this as completed May 8, 2015
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

No branches or pull requests

1 participant