https://bz.apache.org/bugzilla/show_bug.cgi?id=65301

--- Comment #3 from Remy Maucherat <r...@apache.org> ---
As it is right now, the enableLookups flags documentation is accurate: "Set to
true if you want calls to request.getRemoteHost() to perform DNS lookups in
order to return the actual host name of the remote client. Set to false to skip
the DNS lookup and return the IP address in String form instead (thereby
improving performance). By default, DNS lookups are disabled."

As a result, I think the current behavior of the flag should not be changed. If
the local name lookup is really a problem, then a new flag can be introduced.
Reading the RemoteIpValve code, saving and restoring the localName is not
needed if isChangeLocalName() is false (the default value). Since this has a
cost, it's an easy enhancement that I'll do.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to