Content-Length header duplicated on Flash #18046
Labels
bug
Something isn't working correctly
Comments
levex
pushed a commit
that referenced
this issue
Apr 12, 2023
This commit changes implementation of "Deno.serve()" API to use "Deno.serveHttp()" under the hood. This change will allow us to remove the "flash" server implementation, bringing stability to the "Deno.serve()" API. "cli/tests/unit/flash_test.ts" was renamed to "serve_test.ts". Closes #15574 Closes #15504 Closes #15646 Closes #15909 Closes #15911 Closes #16828 Closes #18046 Closes #15869
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When sending a
Responsewith the Flash server, and the user provides aContent-Lengthheader, the Flash server will add an additionalContent-Lengthheader with the content length as it calculated it, where as if theContent-Lengthis provided to the "native" stable server, it does not duplicate the header (I am not sure if it is overriding it or if it is just letting it pass through).It feels like the Flash server should follow the behaviour of the native stable server.
Ref: oakserver/oak#587 (I will fix it in oak by simply omitting the
content-lengthin all cases as the servers are actually better at handling this header now)The text was updated successfully, but these errors were encountered: