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.

Sign up for GitHub

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

Jump to bottom

Unexpected response from a Controller using tomcat #177

Closed
3 tasks done
dcalap opened this issue Apr 13, 2021 · 0 comments · Fixed by #184
Closed
3 tasks done

Unexpected response from a Controller using tomcat #177

dcalap opened this issue Apr 13, 2021 · 0 comments · Fixed by #184
Labels
status: pr submitted A pull request has been submitted for the issue type: bug Something isn't working

Comments

@dcalap
Copy link

dcalap commented Apr 13, 2021

Task List

  • Steps to reproduce provided
  • [-] Stacktrace (if present) provided
  • Example that reproduces the problem uploaded to Github
  • Full description of the issue provided (see below)

Steps to Reproduce

  1. Run application with ./gradlew run -Pmicronaut.runtime=tomcat
  2. Perform a POST request (curl, httpie...) against a micronaut application controller and check the result (http status code)

Expected Behaviour

It should answer with a 200

Actual Behaviour

It's answering with a 404 but still processing the request

issue

Environment Information

  • Operating System: Ubuntu 20.04.2 LTS
  • Micronaut Version: 2.4.2
  • JDK Version: JDK 11.0.8

Example Application

https://github.com/dcalap/mn-tomcat-issue
@graemerocher graemerocher transferred this issue from micronaut-projects/micronaut-core Apr 13, 2021
graemerocher added a commit that referenced this issue Apr 14, 2021
@graemerocher
Don't set response to 404 when returning an http response. Fixes #177
0dad009
@graemerocher graemerocher added status: pr submitted A pull request has been submitted for the issue type: bug Something isn't working labels Apr 14, 2021
@sdelamo sdelamo linked a pull request Apr 15, 2021 that will close this issue
Don't set response to 404 when returning an http response. Fixes #177 #178
Closed
sdelamo pushed a commit that referenced this issue Apr 15, 2021
@graemerocher @sdelamo
Don't set response to 404 when returning an http response. Fixes #177
1725275
@sdelamo sdelamo mentioned this issue Apr 15, 2021
Merged
@sdelamo sdelamo linked a pull request Apr 15, 2021 that will close this issue
Don't set response to 404 when returning an http response #184
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status: pr submitted A pull request has been submitted for the issue type: bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@graemerocher @dcalap