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
Panicking inside "replace_trailing_slash" function #1122
Comments
I'm curious whether this is a bug in the |
* Fix possible panic when doing trailing slash redirect Fixes #1122 * changelog link * Update axum/src/routing/mod.rs Co-authored-by: Jonas Platte <jplatte+git@posteo.de> Co-authored-by: Jonas Platte <jplatte+git@posteo.de>
Was this bringing down the server, or was unwinding caught? |
It wouldn't bring down the server. Just close the connection without sending a response, because hyper (or maybe tokio) catches the panic. The fix has been released in 0.5.10 🎉 |
FYI this was indeed bringing down my server, but I guess this is probably because I was aborting upon panicking |
Yes. If you do that there is nothing tokio/hyper can do. |
Bug Report
Version
0.5.7
Platform
Linux prod 5.10.0-9-amd64 #1 SMP Debian 5.10.70-1 (2021-09-30) x86_64 GNU/Linux
Description
Axum is panicking here: https://github.com/tokio-rs/axum/blob/main/axum/src/routing/mod.rs#L532
The text was updated successfully, but these errors were encountered: