Github user necouchman commented on a diff in the pull request:
https://github.com/apache/incubator-guacamole-client/pull/109#discussion_r97834210
--- Diff: guacamole/src/main/java/org/apache/guacamole/rest/APIRequest.java
---
@@ -58,6 +68,27 @@ public APIRequest(HttpServletRequest request,
super(request);
+ // Try a few methods to get client info.
+ if(request.getHeader("X-Guacamole-Client-Hostname") != null &&
request.getHeader("X-Guacamole-Client-Hostname") != "") {
--- End diff --
Depending on how Guacamole + Tomcat is configured, namely behind a proxy,
it can be difficult to get the original client information through. By
default, the remote host/IP info when Tomcat is behind an Apache httpd or nginx
server is whatever the hostname and IP of the proxy is, rather than the client.
This is not the intended behavior of the module, since you want the hostname
and/or IP of the actual client, not the proxy server. I implemented it this
way in order to give some options - you can set up custom headers that pass
through, you can do some Apache+Tomcat configuration, or you can rely on the
standard remote host/addr fields. See comments on the JIRA issue:
https://issues.apache.org/jira/browse/GUACAMOLE-47
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
