feat: handle HTTP 413 error in upload [1 day]#11115
Merged
sissbruecker merged 2 commits intomainfrom Feb 17, 2026
Merged
Conversation
Add support for HTTP 413 (Payload Too Large) status code in the upload component. - Add error handling for 413 status in UploadManager - Add i18n configuration and error mapping in UploadMixin - Update TypeScript definitions for new error type - Add unit tests for 413 error handling - Integrate 413 testing into dev/upload.html demo - Make nodrop upload respect same configuration Closes #11111
Member
Author
|
Vibed with Mistral Vibe, but verified fix manually. NOT tested with a real server or a reverse proxy giving the error code, but looks I'm rather confident it works. Might be good to improve the platform/java component setups to have e.g. ngingx based test setup with a reverse proxy, but probably out of the scope of this change. |
Member
Author
|
Does the i18n message need to be extended to the Java side 🤔 |
yuriy-fix
changed the title
yuriy-fix
changed the title
|
sissbruecker
approved these changes
Feb 17, 2026
Collaborator
|
This ticket/PR has been released with Vaadin 25.1.0-alpha9. |
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add support for HTTP 413 (Payload Too Large) status code in the upload component.
Add error handling for 413 status in UploadManager
Add i18n configuration and error mapping in UploadMixin
Update TypeScript definitions for new error type
Add unit tests for 413 error handling
Integrate 413 testing into dev/upload.html demo
Make nodrop upload respect same configuration
Fixes #11111
Type of change