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.

Sign up for GitHub

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

Jump to bottom

fix: CORS headers #59

Merged

alanshaw merged 1 commit into main from fix/cors-headers

May 15, 2024

Member

alanshaw commented May 15, 2024

Vary may be set already by handlers, so use append instead
Added HEAD to allowed methods
Removed Content-Length from expose headers as it is superfluous - it is on the CORS response header safelist
Added Content-Range to expose headers since we now support range requests


          fix: CORS headers

d30b211

alanshaw requested review from Gozala and vasco-santos

May 15, 2024 14:18

vasco-santos approved these changes

View reviewed changes

alanshaw merged commit 1726f8b into main

1 check passed

alanshaw deleted the fix/cors-headers branch

May 15, 2024 14:27

github-actions bot mentioned this pull request

chore(main): release 5.0.1 #60

Merged

alanshaw pushed a commit that referenced this pull request


          chore(main): release 5.0.1 (#60)

7346b2c

🤖 I have created a release *beep* *boop*
---


##
[5.0.1](v5.0.0...v5.0.1)
(2024-05-15)


### Bug Fixes

* CORS headers
([#59](#59))
([1726f8b](1726f8b))
* export http range utility functions
([7cf1e62](7cf1e62))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet