-
Notifications
You must be signed in to change notification settings - Fork 56
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
Random panic with caddy plugin #246
Comments
Hello @ueffel I saw that and already worked on a potential fix about the multiple headers sent. Can you try using the hash |
I will try it out and keep an eye on it. As I said I have no way to consistently reproduce the error, so a definitive answer if its fixed is hard. |
Hmpf it's not even building on my raspberry pi...
|
I can downgrade to v0.9.0 if you want to fix that. |
The hash |
xcaddy still tries to build nutsdb v0.10.0. Possibly because of this line: Line 126 in 275879f
|
You're right, I'll check the root cause it doesn't use the v0.9.0 only |
This one includes the downgrade to
|
No panics yet, but I noticed that cache-hits are not compressed anymore. |
I don't override the |
I think the Edit: Oh I didn't answer your question: the backend response is plain, no encoding. Gzip-Encoding is done by caddy. |
Still getting a panic... |
I'll rewrite the response streaming implementation then |
I reverted everything that could cause the panic in the last commit |
We've seen a lot of similar errors recently in our set up with caddy, it happened under the load, maybe relevant. Do you have some ideas of where we should look in? We use pretty much default configuration (badger). Attaching the stacktrace (stacktrace includes Thanks in advance 🙏
|
@RomanHotsiy can you try with the latest release? (v1.6.50) |
Hey @darkweak. We switched to the latest version and we still can see those random errors. We use it via caddy cache-handler 0.13.0: https://github.com/caddyserver/cache-handler/releases/tag/v0.13.0 We also verified our order:
Looks valid:
Stack trace attached:
|
Did you try with souin v1.6.50 instead? There are lot of improvements in the dev repository that are not used in the cache-handler. |
it seems to be fixed on the 1.6.50! 🎉 Thanks for your prompt replies! |
I see the following panic randomly on my raspberry pi. I don't see a pattern yet. Any ideas?
The text was updated successfully, but these errors were encountered: