Skip to content

Work around CORS issues with jsdom in auth/hub emulators.#3224

Merged
yuchenshi merged 2 commits into
masterfrom
ys/cors-jsdom
Mar 19, 2021
Merged

Work around CORS issues with jsdom in auth/hub emulators.#3224
yuchenshi merged 2 commits into
masterfrom
ys/cors-jsdom

Conversation

@yuchenshi

Copy link
Copy Markdown
Member

Description

Fixes #3200.

Scenarios Tested

See tests added.

Sample Commands

N/A

@google-cla google-cla Bot added the cla: yes Manual indication that this has passed CLA. label Mar 19, 2021
Comment thread package.json
"cli-table": "^0.3.1",
"commander": "^4.0.1",
"configstore": "^5.0.1",
"cors": "^2.8.5",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is tricker than I thought, and I'm glad someone else solved it already. This adds 6KiB download size only, mainly due to vary (but we'd need to handle that anyway). This is also already a dependency in Firebase Functions SDK.

@yuchenshi yuchenshi requested a review from abeisgoat March 19, 2021 19:09
@yuchenshi yuchenshi merged commit cae8fa5 into master Mar 19, 2021
@yuchenshi yuchenshi deleted the ys/cors-jsdom branch March 19, 2021 21:18
devpeerapong pushed a commit to devpeerapong/firebase-tools that referenced this pull request Dec 14, 2021
)

* Work around CORS issues with jsdom in auth/hub emulators.

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

Labels

cla: yes Manual indication that this has passed CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auth emulator fails to create user with X-Client-Version forbidden from jsdom

3 participants