Bug: uploads using form-data do not honour per-chart-limit setting #1050
Assignees
Comments
alexberry
added a commit
to alexberry/chartmuseum
that referenced
this issue
Feb 14, 2025
* Ensure uploads using form-data utilise server.PutWithLimit * Add tests * Fixes helm#1050
alexberry
added a commit
to alexberry/chartmuseum
that referenced
this issue
Feb 14, 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
In #466 , support for per-chart limits was added, however during testing I found a bug.
There are two methods to upload a file to chartmuseum - one using binary data, the other using forms data. We upload our charts using the helm cm-push plugin, and it uses forms data to upload.
The forms data upload code path has not been updated to clean up like it has in the binary code path.
The text was updated successfully, but these errors were encountered: