Skip to content
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

Web Extensions no longer work in dev mode (fix #220171) #221030

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Conversation

bpasero
Copy link
Member

@bpasero bpasero commented Jul 8, 2024

The web worker is now using importScript directive in a data URL which is no longer the same script-src as the iframe when running built where we use webEndpointUrlTemplate. This breaks vscode-test-web that runs from localhost so we allow http://localhost

The web worker is now using `importScript` directive in a data URL which is no longer the same `script-src` as the iframe when running built where we use `webEndpointUrlTemplate`. This breaks `vscode-test-web` that runs from `localhost` so we allow `http://localhost`
@bpasero bpasero self-assigned this Jul 8, 2024
@bpasero bpasero enabled auto-merge (squash) July 8, 2024 10:32
@vscodenpa vscodenpa added this to the July 2024 milestone Jul 8, 2024
@bpasero bpasero merged commit 6f18713 into main Jul 8, 2024
7 checks passed
@bpasero bpasero deleted the ben/tricky-deer branch July 8, 2024 10:48
aaronchucarroll pushed a commit to aaronchucarroll/vscode that referenced this pull request Jul 10, 2024
…rosoft#221030)

The web worker is now using `importScript` directive in a data URL which is no longer the same `script-src` as the iframe when running built where we use `webEndpointUrlTemplate`. This breaks `vscode-test-web` that runs from `localhost` so we allow `http://localhost`
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Aug 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants