-
Notifications
You must be signed in to change notification settings - Fork 237
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
Use :authority
psueod-header on HTTP/2 if available
#657
Comments
I looked into this, The additional problem though is that either Deno does not process the header. |
@kitsonk Thanks for the clarification and the split, nothing to be concerned about. So you say Deno itself doesn't provide the headers? Could you please link me to the relevant code that "simply takes what Deno offers to it"? I would be happy to bubble the issue up, but I couldn't pin-point the exact code where Oak receives the headers from Deno. And yes, you are correct about your findings, and most (if not all) common browsers switched to HTTP/2 as far as I can tell. Thank you! |
The headers on the request are from the server request: Lines 70 to 72 in 77d50d6
And the server request used for Deno uses the headers from the oak/http_server_native_request.ts Lines 65 to 67 in 77d50d6
The indirection is there is because of the evolution of Deno HTTP since 2018 and the need to support multiple runtimes. |
Closed in 0c8b8da |
Originally posted by @martin-braun in #571 (comment)
The text was updated successfully, but these errors were encountered: