-
Notifications
You must be signed in to change notification settings - Fork 18k
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
net/rpc: access to client's IP address #4584
Comments
How come this issue is now considered closed? |
As the record shows, Russ closed the issue as "unfortunate" on July 30, 2013. We use the "unfortunate" status for issues that are the result of a possibly mistaken choice in the past. |
I'm not sure that it's true that this issue is unfixable though? For example, given that the RPC layer already does reflection on the RPC methods of the implementing class, it would not be too hard to have it detect if a method also takes a request argument, and if so, pass it when the method is called? |
"Unfortunate" doesn't mean it's impossible to fix. It means that we've decided it's not worth fixing given the constraints we have, especially that of backwards compatibility. |
Does this issue still exists? |
@andyxning: what do you mean? Yes, this is still absolutely a useful feature. Not sure why it was just closed :/ |
@jonhoo I mean that until now we can not implement this functionality about get remote ip address inside the rpc function. right? |
by visan.ovidiu:
The text was updated successfully, but these errors were encountered: