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

Cross-Origin-Embedder-Policy support #1851

Open
Sora2455 opened this issue Jun 8, 2022 · 5 comments
Open

Cross-Origin-Embedder-Policy support #1851

Sora2455 opened this issue Jun 8, 2022 · 5 comments

Comments

@Sora2455
Copy link

Sora2455 commented Jun 8, 2022

Is your feature request related to a problem? Please describe.
When trying to set the Cross-Origin-Embedder-Policy header to "require-corp" on my site, requests to application insights (in particular https://australiaeast-0.in.applicationinsights.azure.com//v2/track) are blocked, as they do not have a Cross-Origin-Resource-Policy header.

Describe the solution you'd like
The application insights endpoints to serve a Cross-Origin-Resource-Policy header of the value "cross-origin".

Describe alternatives you've considered
If this was an image or script request, I could add the crossOrigin attribute to the or <script> tag to get around this problem. However, this is a fetch/XHR request, and one that requires cookies if I understand correctly.

Additional context
Cross-Origin-Embedder-Policy is needed to enable the security feature cross-site isolation.

@Karlie-777
Copy link
Contributor

Related: #1399

@Sora2455
Copy link
Author

Any chance this could be looked at anytime soon? I appreciate the team behind the JS library may not have any involvement with the team controlling the report servers, but in that case is there a better place I should be making this request?

@Karlie-777
Copy link
Contributor

Hi @Sora2455,
If your are using a customized endpoint, config customerHeaders should be able to add extra headers.
And another way of doing it is to provide a xhroverride in sender config where you can control sending methods including headers as well.

@Sora2455
Copy link
Author

Hi @Karlie-777

If I've understood correctly, those config options set headers on the request - but I need them set on the response.

@Karlie-777
Copy link
Contributor

@Sora2455 Correct, they are for request headers. For response headers, we've created a task internally, let me get back to you as soon as we have updates!

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

No branches or pull requests

3 participants