Skip to content

Limit action request body size#15585

Merged
matthewp merged 2 commits intomainfrom
cherry-522f880b07
Feb 23, 2026
Merged

Limit action request body size#15585
matthewp merged 2 commits intomainfrom
cherry-522f880b07

Conversation

@matthewp
Copy link
Copy Markdown
Contributor

@matthewp matthewp commented Feb 19, 2026

Changes

  • Add a default 1MB limit for action request bodies and enforce when Content-Length is missing
  • Cherry-picked from 522f880

* Limit the size of an action payload

Limits the size to prevent exhausting the server and potentionally crashing it.

* fix build
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 19, 2026

🦋 Changeset detected

Latest commit: 9fa4492

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Feb 19, 2026
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 19, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing cherry-522f880b07 (9fa4492) with main (30cd6db)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (425ea16) during the generation of this report, so 30cd6db was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@qzio
Copy link
Copy Markdown
Contributor

qzio commented Feb 20, 2026

Please consider #15589 or something similar.

@matthewp matthewp marked this pull request as ready for review February 20, 2026 17:31
@matthewp matthewp merged commit 98ea30c into main Feb 23, 2026
41 of 42 checks passed
@matthewp matthewp deleted the cherry-522f880b07 branch February 23, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants