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
Check if route exist before checking Content-Type of body #4286
Conversation
Cant we check before we run the content type parser if we havea 404? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Yes that is possible. I checked with the tests. seems like I was bit wrong in understanding the comment. |
- check if the route is 404 only if the content-parser is undefined by this we can avoid breaking changes. - this also passes previous tests this resolves issue - fastify/fastify-multipart#381
I have updated the check. thanks to @Uzlopak for giving me hint All the tests now passes |
Noted, thanks for the review Co-authored-by: Manuel Spigolon <behemoth89@gmail.com>
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Checklist
npm run test
andnpm run benchmark
and the Code of conduct