-
-
Notifications
You must be signed in to change notification settings - Fork 150
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
(bug) Panic concurrent map writes #2758
Comments
Hey @natenho thanks for raising it, this is a behavior I have never seen. When it comes to Small question, does it kill the relay-proxy or is it staying up and running? |
After looking at it, the issue seems to be in the In the source there is this warning: // WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING
// WARNING: Timeout middleware causes more problems than it solves.
// WARNING: This middleware should be first middleware as it messes with request Writer and could cause data race if
// it is in other position And for now the middleware is not on the 1st position. I will open a PR to move the middleware as the 1st position. |
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
Observed behaviour
Here is the full log:
Expected Behavior
No panics at all
Steps to reproduce
Not easily reproducible
The text was updated successfully, but these errors were encountered: