Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(@angular/ssr): ensure correct referer header handling in web request conversion #30583

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

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Jun 24, 2025

The Referer header is classified as a forbidden header name within the Web Standard Fetch API. This means it cannot be manually set directly within the headers option, instead it must be set when constructing a new Request() object.

Closes #30581

@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Jun 24, 2025
@alan-agius4 alan-agius4 requested a review from jkrems June 24, 2025 07:17
…est conversion

The `Referer` header is classified as a forbidden header name within the Web Standard Fetch API. This means it cannot be manually set directly within the `headers` option, instead it must be set when constructing a `new Request()` object.

Closes angular#30581
@jkrems jkrems added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 24, 2025
@alan-agius4 alan-agius4 merged commit 1c5bd2e into angular:main Jun 24, 2025
31 checks passed
@alan-agius4 alan-agius4 deleted the ssr-referer-header branch June 24, 2025 14:25
@alan-agius4
Copy link
Collaborator Author

The changes were merged into the following branches: main, 20.0.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: @angular/ssr target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SSR REQUEST token no longer holds the referer header
2 participants