-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add option for middleware to set custom remote address #1009
Add option for middleware to set custom remote address #1009
Conversation
lint failure does not seem to be related to my PR :) |
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.
It's probably best to also set ctx.remoteAddr = nil
in RequestCtx.Init2
.
ok, will add. Should I remove that from |
No keep it there as well as the normal flow doesn't use Init. |
@erikdubbelboer fixed |
Thanks! |
Thanks for prompt review |
would be nice to tag version so that it would be easier to use it :) |
Done |
Fixes #1008