Skip to content

server: add support for graceful shutdown #139

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Bogdanp
Copy link
Contributor

@Bogdanp Bogdanp commented Jul 19, 2025

This change adds a new safety limit, #:shutdown-grace-period, which controls how long the server will wait for in-progress connections to drain before shutting down. The current behaviour of shutting down immediately is preserved by default, so this change is opt-in.

@Bogdanp Bogdanp force-pushed the master branch 5 times, most recently from 4b2e41b to 6be4684 Compare July 19, 2025 07:36
@Bogdanp
Copy link
Contributor Author

Bogdanp commented Jul 19, 2025

I think the "gh3" failure is due to an incompatibility between old versions of Racket and new versions of OpenSSL. After some experimenting, I got the following error: https://github.com/Bogdanp/web-server/actions/runs/16388725990/job/46311858304#step:6:97

I'll push a change to add some more recent versions of Racket to the test matrix and to disable that test for older versions.

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.

1 participant