Skip to content

docs: add documentation for HttpClient response body size limit and r…#69432

Open
SkyZeroZx wants to merge 1 commit into
angular:mainfrom
SkyZeroZx:add-docs-http-error-ssr
Open

docs: add documentation for HttpClient response body size limit and r…#69432
SkyZeroZx wants to merge 1 commit into
angular:mainfrom
SkyZeroZx:add-docs-http-error-ssr

Conversation

@SkyZeroZx

Copy link
Copy Markdown
Contributor

Add documentation for HttpClient response body size limit and related error NG02825

Also partially related to #69388

@pullapprove pullapprove Bot requested a review from JeanMeche June 18, 2026 22:34
@angular-robot angular-robot Bot added the area: docs Related to the documentation label Jun 18, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jun 18, 2026
@SkyZeroZx SkyZeroZx force-pushed the add-docs-http-error-ssr branch from d02516a to 1340e2f Compare June 19, 2026 01:04

const serverConfig: ApplicationConfig = {
providers: [
provideServerRendering({maxResponseBodySize: 5 * 1024 * 1024}, withRoutes(serverRoutes)),

@JeanMeche JeanMeche Jun 19, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: to make it obvious why we do this computation

Suggested change
provideServerRendering({maxResponseBodySize: 5 * 1024 * 1024}, withRoutes(serverRoutes)),
// maxResponseBodySize: 5MB
provideServerRendering({maxResponseBodySize: 5 * 1024 * 1024}, withRoutes(serverRoutes)),

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@SkyZeroZx SkyZeroZx force-pushed the add-docs-http-error-ssr branch from 1340e2f to aa13f9d Compare June 19, 2026 14:58
@SkyZeroZx SkyZeroZx force-pushed the add-docs-http-error-ssr branch from aa13f9d to e5098f0 Compare June 19, 2026 14:59
@pullapprove pullapprove Bot requested review from JeanMeche and kirjs June 19, 2026 15:05

@JeanMeche JeanMeche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewed-for: public-api

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels Jun 19, 2026
@JeanMeche

Copy link
Copy Markdown
Member

Caretaker note: this is a doc only change, no need to presubmit.

@JeanMeche JeanMeche removed the request for review from kirjs June 19, 2026 15:06
@JeanMeche JeanMeche added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Jun 19, 2026
@JeanMeche

Copy link
Copy Markdown
Member

We'll need a patch backport, as it conflictds with the 22.0 branch

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: docs Related to the documentation merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants