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

chore: fixes and cleanup #22

Merged
merged 6 commits into from
Dec 2, 2023
Merged

chore: fixes and cleanup #22

merged 6 commits into from
Dec 2, 2023

Conversation

jdockerty
Copy link
Owner

@jdockerty jdockerty commented Dec 2, 2023

  • feat: optional graceful shutdown
  • fix: response headers dependent on request, rather than being statically defined.
  • update readme performance showcase

@jdockerty jdockerty self-assigned this Dec 2, 2023
The trailing \r\n meant that the content-length was being seen as wrong which caused errors on reading back the value, shaving off responses too preemptively
@jdockerty jdockerty merged commit c883cc2 into main Dec 2, 2023
1 check passed
@jdockerty jdockerty deleted the fixup branch December 2, 2023 17:31
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.

None yet

1 participant