Skip to content

[browser] improve CSP #86158

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
Merged
lewing merged 1 commit into from
May 12, 2023
Merged

[browser] improve CSP #86158

lewing merged 1 commit into from
May 12, 2023

Conversation

@pavelsavara
Copy link
Member

@pavelsavara pavelsavara commented May 12, 2023

  • removed wrap_trace_function which was eval() and @kg said she didn't use it for long time
  • added CSP meta into Advanced Sample to test that we are compliant.
  • fixed fetch replacement in advanced sample
  • some white space formatting

Contributes to dotnet/aspnetcore#37787

@pavelsavara pavelsavara added this to the 8.0.0 milestone May 12, 2023
@pavelsavara pavelsavara self-assigned this May 12, 2023
@pavelsavara pavelsavara requested review from kg and lewing as code owners May 12, 2023 16:09
@ghost
Copy link

ghost commented May 12, 2023

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details
  • removed wrap_trace_function which was eval() and @kg said she didn't use it for long time
  • added CSP meta into Advanced Sample to test that we are compliant.
  • fixed fetch replacement in advanced sample
  • some white space formatting

Contributes to dotnet/aspnetcore#37787

Author: pavelsavara
Assignees: pavelsavara
Labels:

arch-wasm, area-System.Runtime.InteropServices.JavaScript
Milestone: 8.0.0

@lewing lewing mentioned this pull request May 12, 2023
Closed
@lewing lewing merged commit 99bb1b8 into dotnet:main May 12, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Jun 12, 2023
@pavelsavara pavelsavara deleted the browser_csp branch September 2, 2024 15:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@lewing lewing lewing approved these changes

@kg kg kg approved these changes

Assignees

@pavelsavara pavelsavara

Labels

arch-wasm WebAssembly architecture area-System.Runtime.InteropServices.JavaScript

Projects

None yet

Milestone

8.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@pavelsavara @lewing @kg