Skip to content

Fix per-chart limits for form-data based uploads.#1051

Merged
scbizu merged 1 commit into
helm:mainfrom
alexberry:fix-per-chart-limits-form-data
Mar 14, 2025
Merged

Fix per-chart limits for form-data based uploads.#1051
scbizu merged 1 commit into
helm:mainfrom
alexberry:fix-per-chart-limits-form-data

Conversation

@alexberry

@alexberry alexberry commented Feb 14, 2025

Copy link
Copy Markdown
Contributor

@alexberry
alexberry force-pushed the fix-per-chart-limits-form-data branch 2 times, most recently from abe9b8f to 36a40c0 Compare February 14, 2025 14:54
@scbizu

scbizu commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

Thank you , I'll take a look at this

@scbizu

scbizu commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

Hi @alexberry , could you somehow verify your commits ?

@scbizu scbizu self-assigned this Feb 20, 2025
@scbizu scbizu added the bug label Feb 20, 2025
@scbizu scbizu mentioned this pull request Mar 11, 2025
@alexberry
alexberry force-pushed the fix-per-chart-limits-form-data branch 6 times, most recently from e04c9a6 to 88e061b Compare March 12, 2025 11:00
@alexberry

Copy link
Copy Markdown
Contributor Author

Hi @alexberry , could you somehow verify your commits ?

hi @scbizu, done & rebased :)

@alexberry
alexberry force-pushed the fix-per-chart-limits-form-data branch from 88e061b to 67b47ce Compare March 12, 2025 11:01
* Ensure uploads using form-data utilise server.PutWithLimit
* Add tests
* Fixes helm#1050

Signed-off-by: alexberry <[email protected]>
@alexberry
alexberry force-pushed the fix-per-chart-limits-form-data branch from 67b47ce to 63464a1 Compare March 12, 2025 11:02
@scbizu
scbizu merged commit dee42f2 into helm:main Mar 14, 2025
nerdeveloper pushed a commit to nerdeveloper/chartmuseum that referenced this pull request May 18, 2025
* Ensure uploads using form-data utilise server.PutWithLimit
* Add tests
* Fixes helm#1050

Signed-off-by: alexberry <[email protected]>
Signed-off-by: Obinna Odirionye <[email protected]>
dmytrop-salt pushed a commit to Secful/chartmuseum that referenced this pull request Oct 13, 2025
* Ensure uploads using form-data utilise server.PutWithLimit
* Add tests
* Fixes helm#1050

Signed-off-by: alexberry <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: uploads using form-data do not honour per-chart-limit setting

2 participants