diff options
author | 2023-06-06 11:09:16 -0400 | |
---|---|---|
committer | 2023-06-06 10:09:16 -0500 | |
commit | 4929332c3210d1634b8607c7736d9049860a2079 (patch) | |
tree | f232675fb2d6820e6262a0f50aaa49efa42e9e0e /examples/docs/src | |
parent | 409c60028aaab09b8f2383ef5730531cd23db4ba (diff) | |
download | astro-4929332c3210d1634b8607c7736d9049860a2079.tar.gz astro-4929332c3210d1634b8607c7736d9049860a2079.tar.zst astro-4929332c3210d1634b8607c7736d9049860a2079.zip |
#7226 - fixes NodeJS adapter for multiple set-cookie headers (and other header issues) (#7227)
* Utilizes the new standard WebAPI Fetch Headers.getSetCookie() function
to safely handle multiple set-cookie headers when converting from a
WebAPI Response to a NodeJS ServerResponse
Modifies the existing nodeMiddleware logic which first set AstroCookies
on ServerResponse.setHeader(...) and then called
ServerResponse.writeHead(status, Response.headers) which means any that
if the WebAPI Response had any set-cookie headers on it, they would
replace anything from AstroCookies.
The new logic delegates appending AstroCookie values onto the WebAPI
Response Headers object, so that a single unified function safely
converts the WebAPI Response Headers into a NodeJS compatible
OutgoingHttpHeaders object utilizing the new standard
Headers.getSetCookie() function provided by the undici WebAPI polyfills.
Plus extensive test coverage.
* #7226 - changeset for NodeJS adapter set-cookie fix
* fixing all double quotes to single quotes
---------
Co-authored-by: Alex Sherwin <alex.sherwin@acadia.inc>
Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>
Diffstat (limited to 'examples/docs/src')
0 files changed, 0 insertions, 0 deletions