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
ISPN-6545 Avoid leaking HR server req details #4266
Conversation
@@ -49,7 +51,7 @@ public NettyTransport getTransport() { | |||
} | |||
|
|||
void resetNow() { | |||
decodeCtx.resetParams(); |
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.
We may as well remove this method while we are at it.
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.
Sure
Changes are fine with me, but these are also the ones I suggested. If anyone wants to comment please do otherwise I will integrated tomorrow. |
Changes looks good, but is the node.js client the only way to reproduce it? |
@gustavonalle unfortunately yes. It only affected an async client, where it was sending multiple requests on the same socket. |
@gustavonalle We'd need an async Java client to replicate indeed. |
@wburns Updated |
Pulling.. |
Integrated into master, thanks @galderz ! |
https://issues.jboss.org/browse/ISPN-6545