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

Added default headers for chromium in direct requests #1765

Merged

ninadbstack merged 1 commit into master from add-default-headers-in-direct-requests

Oct 24, 2024

Contributor

ninadbstack commented Oct 24, 2024

Context:

It seems like some security softwares / or cdns have started requiring these headers - else they return 403s
I am assuming that addition of extra headers would not affect where its already working


          Added default headers for chromium in direct requests

f033bd6

ninadbstack requested a review from a team as a code owner

October 24, 2024 08:05

ninadbstack requested review from pankaj443 and shantanuk-browserstack

October 24, 2024 08:05

prklm10 approved these changes

View reviewed changes

Contributor

prklm10 left a comment

approved. As this request is made only for font. So this should not affect other calls.

ninadbstack merged commit c0e5870 into master

36 checks passed

ninadbstack deleted the add-default-headers-in-direct-requests branch

October 24, 2024 08:22

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

Reviewers

prklm10 prklm10 approved these changes

pankaj443 Awaiting requested review from pankaj443 pankaj443 is a code owner automatically assigned from percy/percy-product-reviewers

shantanuk-browserstack Awaiting requested review from shantanuk-browserstack shantanuk-browserstack is a code owner automatically assigned from percy/percy-product-reviewers

Labels

None yet