Skip to content

adds --isolate for cross origin isolation headers #759

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rwaldin
Copy link

@rwaldin rwaldin commented Oct 28, 2021

Adds an --isolate command line flag to enable Cross Origin Isolation headers that enable SharedArrayBuffer transfers and use of high precision timers. See https://web.dev/cross-origin-isolation-guide/

Contributor checklist
  • Provide tests for the changes (unless documentation-only)
  • Documented any new features, CLI switches, etc. (if applicable)
    • Server --help output
    • README.md
    • doc/http-server.1 (use the same format as other entries)
  • The pull request is being made against the master branch
Maintainer checklist
  • Assign a version triage tag
  • Approve tests if applicable

@rwaldin
Copy link
Author

rwaldin commented Oct 28, 2021

oops, i see there's already a PR for exactly this. I'll gladly retract this if that other solution is preferred.

@HKalbasi
Copy link

HKalbasi commented Jan 8, 2022

Fixes #650

@charlieroberts
Copy link

please merge! would be super useful and is necessary while custom headers are not enabled.

alxndrsn pushed a commit to alxndrsn/pouchdb that referenced this pull request Apr 2, 2024
See: https://developer.mozilla.org/en-US/docs/Web/API/Performance/now#security_requirements

This requires a change of dependencies from http-server to serve as http-server does not currently support setting the required headers.  See:

* http-party/http-server#759
* http-party/http-server#806
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants