Skip to content
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

Feature#543 #555

Closed
wants to merge 2 commits into from
Closed

Feature#543 #555

wants to merge 2 commits into from

Conversation

ecs90
Copy link
Collaborator

@ecs90 ecs90 commented Jun 27, 2024

Test to verify if payload is too large.

Added test, properties and exception handler

changed spring.servlet and server.tomcat to take a max payload = 2MB
added the exception handler to this too
@ecs90 ecs90 assigned ecs90 and unassigned ecs90 Jun 27, 2024
@ecs90 ecs90 added Testing Testing ita-user micro ita-user micro Level 1 Level 1 labels Jun 27, 2024
@ecs90 ecs90 self-assigned this Jun 27, 2024
Copy link

sonarcloud bot commented Jul 17, 2024

@jonatanvicente
Copy link
Collaborator

jonatanvicente commented Jul 17, 2024

It doesn't work.
Config options added must be working (with both verbs PUT / POST), but a manual test allows send bigger request (attached screenshot).
May be necessary other options. In fact, config option "max-http-form-post-size" added is referring to the data encoded as key-value pairs sent, and represents the input fields of a form.... it seems it doesn't be what we need.

Pasted image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ita-user micro ita-user micro Level 1 Level 1 Testing Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants