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

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

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

Merged
atscott merged 1 commit into
angular:21.2.xfrom
alan-agius4:deprecate-server-xhr-21-2
Jun 9, 2026
Merged

refactor(platform-server): deprecate ServerXhr#69255
atscott merged 1 commit into
angular:21.2.xfrom
alan-agius4:deprecate-server-xhr-21-2

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 kirjs 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 kirjs June 9, 2026 09:09
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 labels Jun 9, 2026
Comment thread packages/platform-server/src/http.ts Outdated
alan-agius4 requested a review from JeanMeche June 9, 2026 09:40
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-21-2 branch from b1770ea to 8717e56 Compare June 9, 2026 09:42
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
alan-agius4 removed the request for review from AndrewKushnir June 9, 2026 09:57
atscott merged commit 13fb0af into angular:21.2.x Jun 9, 2026
20 checks passed

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 Jul 10, 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.

3 participants


Back | FazBrowse Home | New Git URL