Skip to content

[UNDERTOW-2556] fail POST expect if content length exceeds the limit #1727

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aogburn
Copy link
Contributor

@aogburn aogburn commented Jun 5, 2025

@fl4via fl4via added the bug fix Contains bug fix(es) label Jun 11, 2025
@fl4via fl4via added the next release This PR will be merged before next release or has already been merged (for payload double check) label Jun 11, 2025
@fl4via fl4via requested a review from ropalka June 11, 2025 22:12
@fl4via fl4via added the waiting PR update Awaiting PR update(s) from contributor before merging label Jun 11, 2025
@fl4via
Copy link
Member

fl4via commented Jun 11, 2025

As discussed with @aogburn there is another possibility for moving the check to an even sooner point, so I added the Waiting PR update label to indicate that this one is not ready yet.

@aogburn
Copy link
Contributor Author

aogburn commented Jun 16, 2025

Thanks @fl4via . Made an additional change to fail earlier. In this case, the thrown exception will be caught by https://github.com/undertow-io/undertow/blob/main/core/src/main/java/io/undertow/server/protocol/http/HttpReadListener.java#L267 for a 400.

@fl4via fl4via removed the waiting PR update Awaiting PR update(s) from contributor before merging label Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix Contains bug fix(es) next release This PR will be merged before next release or has already been merged (for payload double check)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants