Skip to content

Conversation

@julien-nc
Copy link
Member

refs #39924

Adding this header systematically is breaking any usage of the MapLibre map library for example because the lib reaches endpoints with a strict CORS policy which does not accept extra headers.

@julien-nc julien-nc added bug 3. to review Waiting for reviews labels Sep 18, 2023
@julien-nc julien-nc added this to the Nextcloud 28 milestone Sep 18, 2023
@juliusknorr
Copy link
Member

Conflicts on dist bundles ;)

@julien-nc julien-nc force-pushed the enh/fix-x-requested-with branch from fb6b5ae to f2b010e Compare October 12, 2023 13:37
@julien-nc julien-nc enabled auto-merge October 12, 2023 13:37
@julien-nc julien-nc force-pushed the enh/fix-x-requested-with branch from f2b010e to b58c3dd Compare October 16, 2023 08:29
@julien-nc julien-nc disabled auto-merge October 16, 2023 10:05
@julien-nc
Copy link
Member Author

Cypress failures are unrelated (theming and user settings).

@julien-nc julien-nc merged commit 9bf824e into master Oct 16, 2023
@julien-nc julien-nc deleted the enh/fix-x-requested-with branch October 16, 2023 10:06
@juliusknorr
Copy link
Member

Small follow up fix is in #40944

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

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants