FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

refactor(platform-server): deprecate ServerXhr by alan-agius4 · Pull Request #69256 · angular/angular · GitHub

refactor(platform-server): deprecate ServerXhr - #69256

Merged
atscott merged 1 commit into
angular:20.3.xfrom
alan-agius4:deprecate-server-xhr-20-3
Jun 9, 2026
Merged

refactor(platform-server): deprecate ServerXhr#69256
atscott merged 1 commit into
angular:20.3.xfrom
alan-agius4:deprecate-server-xhr-20-3

Conversation

Copy link
Copy Markdown
Contributor

refactor(platform-server): deprecate ServerXhr

XHR support in @angular/platform-server is deprecated because the underlying xhr2 library does not safely handle redirects. Specifically, it can forward Authorization headers on cross-origin redirects (which leaks credentials) and is susceptible to denial-of-service (DoS) via redirect loops.

DEPRECATED: XHR support in @angular/platform-server is deprecated. Use standard fetch APIs instead.

pullapprove Bot requested a review from atscott June 9, 2026 08:56
angular-robot Bot added detected: deprecation PR contains a commit with a deprecation area: server Issues related to server-side rendering labels Jun 9, 2026
ngbot Bot added this to the Backlog milestone Jun 9, 2026
alan-agius4 requested review from AndrewKushnir and removed request for atscott June 9, 2026 09:08
alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: lts This PR is targeting a version currently in long-term support and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 9, 2026
XHR support in `@angular/platform-server` is deprecated because the underlying `xhr2` library does not safely handle redirects. Specifically, it can forward `Authorization` headers on cross-origin redirects (which leaks credentials) and is susceptible to denial-of-service (DoS) via redirect loops.

DEPRECATED: XHR support in `@angular/platform-server` is deprecated. Use standard `fetch` APIs instead.
alan-agius4 force-pushed the deprecate-server-xhr-20-3 branch from bd03f69 to 0af757c Compare June 9, 2026 09:43
alan-agius4 requested review from JeanMeche and removed request for AndrewKushnir June 9, 2026 09:58
alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 9, 2026
alan-agius4 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 9, 2026
atscott merged commit d55c94a into angular:20.3.x Jun 9, 2026
21 checks passed

Copy link
Copy Markdown
Contributor

@alan-agius4
Q: Would this case qualify for a CVE, or would it not apply? It's similar to GHSA-qxh6-94w6-9r5p.

Also, I still haven't received a concrete answer on https://issuetracker.google.com/u/1/issues/516495362

Copy link
Copy Markdown

This pull request has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

angular-automatic-lock-bot Bot locked and limited conversation to collaborators Aug 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: server Issues related to server-side rendering detected: deprecation PR contains a commit with a deprecation target: lts This PR is targeting a version currently in long-term support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL