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

Known issues with the Kestrel engine integration #545

Open
1 of 11 tasks
Kaliumhexacyanoferrat opened this issue Nov 8, 2024 · 0 comments
Open
1 of 11 tasks

Known issues with the Kestrel engine integration #545

Kaliumhexacyanoferrat opened this issue Nov 8, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@Kaliumhexacyanoferrat
Copy link
Owner

Kaliumhexacyanoferrat commented Nov 8, 2024

This issue allows us to track the issues that still exist with the Kestrel server engine integration (as they were found in #526):

  • Kestrel does not allow to set a custom HTTP status
  • Kestrel does not allow to read the request body twice
  • Kestrel does not allow to read the length of the request body
  • Websockets are not supported as the underlying socket is not exposed by Kestrel

Additionally, the following differences were found:

  • Kestrel does not generate a Keep-Alive header for HTTP/1.1, so we can remove this as well
  • Kestrel uses lower-case for cookie properties
  • Enable HTTP/2 and HTTP/3 with Kestrel #554
  • Kestrel keeps multiple slashes in the url, Kestrel does not (see TestMultipleSlashes)
  • Kestrel replaces + with a blank in an URL, GenHTTP does not (see TestQueryWithSpaces)
  • We need to allow to set path and secureOnly for cookies (already adressed as an issue)
  • GenHTTP sends Content-Length: 0 for an empty response, Kestrel does not
@Kaliumhexacyanoferrat Kaliumhexacyanoferrat added the bug Something isn't working label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant