-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
🐞 Route with no match returns 200 #430
Comments
Thanks for opening your first issue here! 🎉 Be sure to follow the issue template! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord |
Hi @mewben, thanks for your detailed report. Your first We are currently adding good test cases to make sure we won't break anything when we change this. |
Fiber version/commit
v1.10.1
Issue description
Please see sample code below. It returns 200 for any route. I tried removing all the middleware, it now correctly returns 404. Perhaps there's a breaking change in the way middleware is implemented?
Expected behavior
Should it return 404 instead? or is this the intended behaviour?
Steps to reproduce
Any path e.g. localhost:3000/any-path returns 200
Try removing the
app.Use
blocks, it correctly returns 404Code snippet
The text was updated successfully, but these errors were encountered: