https://issues.apache.org/bugzilla/show_bug.cgi?id=47330
authormarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Thu, 5 Nov 2009 02:03:12 +0000 (02:03 +0000)
committermarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Thu, 5 Nov 2009 02:03:12 +0000 (02:03 +0000)
Add RemoteIpValve
Patch provided by Cyrille Le Clerc

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@832974 13f79535-47bb-0310-9956-ffa450edef68

java/org/apache/catalina/valves/LocalStrings.properties
java/org/apache/catalina/valves/RemoteIpValve.java [new file with mode: 0644]
java/org/apache/catalina/valves/mbeans-descriptors.xml
test/org/apache/catalina/valves/RemoteIpValveTest.java [new file with mode: 0644]
webapps/docs/config/valve.xml

index abc4fb8..7252cc6 100644 (file)
@@ -41,6 +41,9 @@ errorReportValve.rootCause=root cause
 errorReportValve.note=note
 errorReportValve.rootCauseInLogs=The full stack trace of the root cause is available in the {0} logs.
 
+# Remote IP valve
+remoteIpValve.syntax=Invalid regular expressions [{0}] provided.
+
 # HTTP status reports
 http.100=The client may continue ({0}).
 http.101=The server is switching protocols according to the "Upgrade" header ({0}).
diff --git a/java/org/apache/catalina/valves/RemoteIpValve.java b/java/org/apache/catalina/valves/RemoteIpValve.java
new file mode 100644 (file)
index 0000000..44a3cff
--- /dev/null
@@ -0,0 +1,712 @@
+/*\r
+ * Licensed to the Apache Software Foundation (ASF) under one or more\r
+ * contributor license agreements.  See the NOTICE file distributed with\r
+ * this work for additional information regarding copyright ownership.\r
+ * The ASF licenses this file to You under the Apache License, Version 2.0\r
+ * (the "License"); you may not use this file except in compliance with\r
+ * the License.  You may obtain a copy of the License at\r
+ * \r
+ *      http://www.apache.org/licenses/LICENSE-2.0\r
+ * \r
+ * Unless required by applicable law or agreed to in writing, software\r
+ * distributed under the License is distributed on an "AS IS" BASIS,\r
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+ * See the License for the specific language governing permissions and\r
+ * limitations under the License.\r
+ */\r
+\r
+package org.apache.catalina.valves;\r
+\r
+import java.io.IOException;\r
+import java.util.ArrayList;\r
+import java.util.Iterator;\r
+import java.util.LinkedList;\r
+import java.util.List;\r
+import java.util.regex.Pattern;\r
+import java.util.regex.PatternSyntaxException;\r
+\r
+import javax.servlet.ServletException;\r
+\r
+import org.apache.tomcat.util.res.StringManager;\r
+import org.apache.catalina.connector.Request;\r
+import org.apache.catalina.connector.Response;\r
+import org.apache.catalina.valves.Constants;\r
+import org.apache.catalina.valves.RequestFilterValve;\r
+import org.apache.catalina.valves.ValveBase;\r
+import org.apache.juli.logging.Log;\r
+import org.apache.juli.logging.LogFactory;\r
+\r
+/**\r
+ * <p>\r
+ * Tomcat port of <a href="http://httpd.apache.org/docs/trunk/mod/mod_remoteip.html">mod_remoteip</a>, this valve replaces the apparent\r
+ * client remote IP address and hostname for the request with the IP address list presented by a proxy or a load balancer via a request\r
+ * headers (e.g. "X-Forwarded-For").\r
+ * </p>\r
+ * <p>\r
+ * Another feature of this valve is to replace the apparent scheme (http/https) and server port with the scheme presented by a proxy or a\r
+ * load balancer via a request header (e.g. "X-Forwarded-Proto").\r
+ * </p>\r
+ * <p>\r
+ * This valve proceeds as follows:\r
+ * </p>\r
+ * <p>\r
+ * If the incoming <code>request.getRemoteAddr()</code> matches the valve's list of internal proxies :\r
+ * <ul>\r
+ * <li>Loop on the comma delimited list of IPs and hostnames passed by the preceding load balancer or proxy in the given request's Http\r
+ * header named <code>$remoteIPHeader</code> (default value <code>x-forwarded-for</code>). Values are processed in right-to-left order.</li>\r
+ * <li>For each ip/host of the list:\r
+ * <ul>\r
+ * <li>if it matches the internal proxies list, the ip/host is swallowed</li>\r
+ * <li>if it matches the trusted proxies list, the ip/host is added to the created proxies header</li>\r
+ * <li>otherwise, the ip/host is declared to be the remote ip and looping is stopped.</li>\r
+ * </ul>\r
+ * </li>\r
+ * <li>If the request http header named <code>$protocolHeader</code> (e.g. <code>x-forwarded-for</code>) equals to the value of\r
+ * <code>protocolHeaderHttpsValue</code> configuration parameter (default <code>https</code>) then <code>request.isSecure = true</code>,\r
+ * <code>request.scheme = https</code> and <code>request.serverPort = 443</code>. Note that 443 can be overwritten with the\r
+ * <code>$httpsServerPort</code> configuration parameter.</li>\r
+ * </ul>\r
+ * </p>\r
+ * <p>\r
+ * <strong>Configuration parameters:</strong>\r
+ * <table border="1">\r
+ * <tr>\r
+ * <th>RemoteIpValve property</th>\r
+ * <th>Description</th>\r
+ * <th>Equivalent mod_remoteip directive</th>\r
+ * <th>Format</th>\r
+ * <th>Default Value</th>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>remoteIPHeader</td>\r
+ * <td>Name of the Http Header read by this valve that holds the list of traversed IP addresses starting from the requesting client</td>\r
+ * <td>RemoteIPHeader</td>\r
+ * <td>Compliant http header name</td>\r
+ * <td>x-forwarded-for</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>internalProxies</td>\r
+ * <td>List of internal proxies ip adress. If they appear in the <code>remoteIpHeader</code> value, they will be trusted and will not appear\r
+ * in the <code>proxiesHeader</code> value</td>\r
+ * <td>RemoteIPInternalProxy</td>\r
+ * <td>Comma delimited list of regular expressions (in the syntax supported by the {@link java.util.regex.Pattern} library)</td>\r
+ * <td>10\.\d{1,3}\.\d{1,3}\.\d{1,3}, 192\.168\.\d{1,3}\.\d{1,3}, 169\.254\.\d{1,3}\.\d{1,3}, 127\.\d{1,3}\.\d{1,3}\.\d{1,3} <br/>\r
+ * By default, 10/8, 192.168/16, 169.254/16 and 127/8 are allowed ; 172.16/12 has not been enabled by default because it is complex to\r
+ * describe with regular expressions</td>\r
+ * </tr>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>proxiesHeader</td>\r
+ * <td>Name of the http header created by this valve to hold the list of proxies that have been processed in the incoming\r
+ * <code>remoteIPHeader</code></td>\r
+ * <td>RemoteIPProxiesHeader</td>\r
+ * <td>Compliant http header name</td>\r
+ * <td>x-forwarded-by</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>trustedProxies</td>\r
+ * <td>List of trusted proxies ip adress. If they appear in the <code>remoteIpHeader</code> value, they will be trusted and will appear\r
+ * in the <code>proxiesHeader</code> value</td>\r
+ * <td>RemoteIPTrustedProxy</td>\r
+ * <td>Comma delimited list of regular expressions (in the syntax supported by the {@link java.util.regex.Pattern} library)</td>\r
+ * <td>&nbsp;</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>protocolHeader</td>\r
+ * <td>Name of the http header read by this valve that holds the flag that this request </td>\r
+ * <td>N/A</td>\r
+ * <td>Compliant http header name like <code>X-Forwarded-Proto</code>, <code>X-Forwarded-Ssl</code> or <code>Front-End-Https</code></td>\r
+ * <td><code>null</code></td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>protocolHeaderHttpsValue</td>\r
+ * <td>Value of the <code>protocolHeader</code> to indicate that it is an Https request</td>\r
+ * <td>N/A</td>\r
+ * <td>String like <code>https</code> or <code>ON</code></td>\r
+ * <td><code>https</code></td>\r
+ * </tr>\r
+ * <tr>\r
+ * </table>\r
+ * </p>\r
+ * <p>\r
+ * <p>\r
+ * This Valve may be attached to any Container, depending on the granularity of the filtering you wish to perform.\r
+ * </p>\r
+ * <p>\r
+ * <strong>Regular expression vs. IP address blocks:</strong> <code>mod_remoteip</code> allows to use address blocks (e.g.\r
+ * <code>192.168/16</code>) to configure <code>RemoteIPInternalProxy</code> and <code>RemoteIPTrustedProxy</code> ; as Tomcat doesn't have a\r
+ * library similar to <a\r
+ * href="http://apr.apache.org/docs/apr/1.3/group__apr__network__io.html#gb74d21b8898b7c40bf7fd07ad3eb993d">apr_ipsubnet_test</a>,\r
+ * <code>RemoteIpValve</code> uses regular expression to configure <code>internalProxies</code> and <code>trustedProxies</code> in the same\r
+ * fashion as {@link RequestFilterValve} does.\r
+ * </p>\r
+ * <hr/>\r
+ * <p>\r
+ * <strong>Sample with internal proxies</strong>\r
+ * </p>\r
+ * <p>\r
+ * RemoteIpValve configuration:\r
+ * </p>\r
+ * <code><pre>\r
+ * &lt;Valve \r
+ *   className="org.apache.catalina.connector.RemoteIpValve"\r
+ *   internalProxies="192\.168\.0\.10, 192\.168\.0\.11"\r
+ *   remoteIPHeader="x-forwarded-for"\r
+ *   remoteIPProxiesHeader="x-forwarded-by"\r
+ *   protocolHeader="x-forwarded-proto"\r
+ *   /&gt;</pre></code>\r
+ * <p>\r
+ * Request values:\r
+ * <table border="1">\r
+ * <tr>\r
+ * <th>property</th>\r
+ * <th>Value Before RemoteIpValve</th>\r
+ * <th>Value After RemoteIpValve</th>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.remoteAddr</td>\r
+ * <td>192.168.0.10</td>\r
+ * <td>140.211.11.130</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-for']</td>\r
+ * <td>140.211.11.130, 192.168.0.10</td>\r
+ * <td>null</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-by']</td>\r
+ * <td>null</td>\r
+ * <td>null</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-proto']</td>\r
+ * <td>https</td>\r
+ * <td>https</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.scheme</td>\r
+ * <td>http</td>\r
+ * <td>https</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.secure</td>\r
+ * <td>false</td>\r
+ * <td>true</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.serverPort</td>\r
+ * <td>80</td>\r
+ * <td>443</td>\r
+ * </tr>\r
+ * </table>\r
+ * Note : <code>x-forwarded-by</code> header is null because only internal proxies as been traversed by the request.\r
+ * <code>x-forwarded-by</code> is null because all the proxies are trusted or internal.\r
+ * </p>\r
+ * <hr/>\r
+ * <p>\r
+ * <strong>Sample with trusted proxies</strong>\r
+ * </p>\r
+ * <p>\r
+ * RemoteIpValve configuration:\r
+ * </p>\r
+ * <code><pre>\r
+ * &lt;Valve \r
+ *   className="org.apache.catalina.connector.RemoteIpValve"\r
+ *   internalProxies="192\.168\.0\.10, 192\.168\.0\.11"\r
+ *   remoteIPHeader="x-forwarded-for"\r
+ *   remoteIPProxiesHeader="x-forwarded-by"\r
+ *   trustedProxies="proxy1, proxy2"\r
+ *   /&gt;</pre></code>\r
+ * <p>\r
+ * Request values:\r
+ * <table border="1">\r
+ * <tr>\r
+ * <th>property</th>\r
+ * <th>Value Before RemoteIpValve</th>\r
+ * <th>Value After RemoteIpValve</th>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.remoteAddr</td>\r
+ * <td>192.168.0.10</td>\r
+ * <td>140.211.11.130</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-for']</td>\r
+ * <td>140.211.11.130, proxy1, proxy2</td>\r
+ * <td>null</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-by']</td>\r
+ * <td>null</td>\r
+ * <td>proxy1, proxy2</td>\r
+ * </tr>\r
+ * </table>\r
+ * Note : <code>proxy1</code> and <code>proxy2</code> are both trusted proxies that come in <code>x-forwarded-for</code> header, they both\r
+ * are migrated in <code>x-forwarded-by</code> header. <code>x-forwarded-by</code> is null because all the proxies are trusted or internal.\r
+ * </p>\r
+ * <hr/>\r
+ * <p>\r
+ * <strong>Sample with internal and trusted proxies</strong>\r
+ * </p>\r
+ * <p>\r
+ * RemoteIpValve configuration:\r
+ * </p>\r
+ * <code><pre>\r
+ * &lt;Valve \r
+ *   className="org.apache.catalina.connector.RemoteIpValve"\r
+ *   internalProxies="192\.168\.0\.10, 192\.168\.0\.11"\r
+ *   remoteIPHeader="x-forwarded-for"\r
+ *   remoteIPProxiesHeader="x-forwarded-by"\r
+ *   trustedProxies="proxy1, proxy2"\r
+ *   /&gt;</pre></code>\r
+ * <p>\r
+ * Request values:\r
+ * <table border="1">\r
+ * <tr>\r
+ * <th>property</th>\r
+ * <th>Value Before RemoteIpValve</th>\r
+ * <th>Value After RemoteIpValve</th>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.remoteAddr</td>\r
+ * <td>192.168.0.10</td>\r
+ * <td>140.211.11.130</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-for']</td>\r
+ * <td>140.211.11.130, proxy1, proxy2, 192.168.0.10</td>\r
+ * <td>null</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-by']</td>\r
+ * <td>null</td>\r
+ * <td>proxy1, proxy2</td>\r
+ * </tr>\r
+ * </table>\r
+ * Note : <code>proxy1</code> and <code>proxy2</code> are both trusted proxies that come in <code>x-forwarded-for</code> header, they both\r
+ * are migrated in <code>x-forwarded-by</code> header. As <code>192.168.0.10</code> is an internal proxy, it does not appear in\r
+ * <code>x-forwarded-by</code>. <code>x-forwarded-by</code> is null because all the proxies are trusted or internal.\r
+ * </p>\r
+ * <hr/>\r
+ * <p>\r
+ * <strong>Sample with an untrusted proxy</strong>\r
+ * </p>\r
+ * <p>\r
+ * RemoteIpValve configuration:\r
+ * </p>\r
+ * <code><pre>\r
+ * &lt;Valve \r
+ *   className="org.apache.catalina.connector.RemoteIpValve"\r
+ *   internalProxies="192\.168\.0\.10, 192\.168\.0\.11"\r
+ *   remoteIPHeader="x-forwarded-for"\r
+ *   remoteIPProxiesHeader="x-forwarded-by"\r
+ *   trustedProxies="proxy1, proxy2"\r
+ *   /&gt;</pre></code>\r
+ * <p>\r
+ * Request values:\r
+ * <table border="1">\r
+ * <tr>\r
+ * <th>property</th>\r
+ * <th>Value Before RemoteIpValve</th>\r
+ * <th>Value After RemoteIpValve</th>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.remoteAddr</td>\r
+ * <td>192.168.0.10</td>\r
+ * <td>untrusted-proxy</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-for']</td>\r
+ * <td>140.211.11.130, untrusted-proxy, proxy1</td>\r
+ * <td>140.211.11.130</td>\r
+ * </tr>\r
+ * <tr>\r
+ * <td>request.header['x-forwarded-by']</td>\r
+ * <td>null</td>\r
+ * <td>proxy1</td>\r
+ * </tr>\r
+ * </table>\r
+ * Note : <code>x-forwarded-by</code> holds the trusted proxy <code>proxy1</code>. <code>x-forwarded-by</code> holds\r
+ * <code>140.211.11.130</code> because <code>untrusted-proxy</code> is not trusted and thus, we can not trust that\r
+ * <code>untrusted-proxy</code> is the actual remote ip. <code>request.remoteAddr</code> is <code>untrusted-proxy</code> that is an IP\r
+ * verified by <code>proxy1</code>.\r
+ * </p>\r
+ */\r
+public class RemoteIpValve extends ValveBase {\r
+    \r
+    /**\r
+     * {@link Pattern} for a comma delimited string that support whitespace characters\r
+     */\r
+    private static final Pattern commaSeparatedValuesPattern = Pattern.compile("\\s*,\\s*");\r
+    \r
+    /**\r
+     * The descriptive information related to this implementation.\r
+     */\r
+    private static final String info = "org.apache.catalina.connector.RemoteIpValve/1.0";\r
+    \r
+    /**\r
+     * Logger\r
+     */\r
+    private static Log log = LogFactory.getLog(RemoteIpValve.class);\r
+    \r
+    /**\r
+     * The StringManager for this package.\r
+     */\r
+    protected static StringManager sm = StringManager.getManager(Constants.Package);\r
+    \r
+    /**\r
+     * Convert a given comma delimited list of regular expressions into an array of compiled {@link Pattern}\r
+     * \r
+     * @return array of patterns (not <code>null</code>)\r
+     */\r
+    protected static Pattern[] commaDelimitedListToPatternArray(String commaDelimitedPatterns) {\r
+        String[] patterns = commaDelimitedListToStringArray(commaDelimitedPatterns);\r
+        List<Pattern> patternsList = new ArrayList<Pattern>();\r
+        for (String pattern : patterns) {\r
+            try {\r
+                patternsList.add(Pattern.compile(pattern));\r
+            } catch (PatternSyntaxException e) {\r
+                throw new IllegalArgumentException(sm.getString("remoteIpValve.syntax", pattern), e);\r
+            }\r
+        }\r
+        return patternsList.toArray(new Pattern[0]);\r
+    }\r
+    \r
+    /**\r
+     * Convert a given comma delimited list of regular expressions into an array of String\r
+     * \r
+     * @return array of patterns (non <code>null</code>)\r
+     */\r
+    protected static String[] commaDelimitedListToStringArray(String commaDelimitedStrings) {\r
+        return (commaDelimitedStrings == null || commaDelimitedStrings.length() == 0) ? new String[0] : commaSeparatedValuesPattern\r
+            .split(commaDelimitedStrings);\r
+    }\r
+    \r
+    /**\r
+     * Convert an array of strings in a comma delimited string\r
+     */\r
+    protected static String listToCommaDelimitedString(List<String> stringList) {\r
+        if (stringList == null) {\r
+            return "";\r
+        }\r
+        StringBuilder result = new StringBuilder();\r
+        for (Iterator<String> it = stringList.iterator(); it.hasNext();) {\r
+            Object element = it.next();\r
+            if (element != null) {\r
+                result.append(element);\r
+                if (it.hasNext()) {\r
+                    result.append(", ");\r
+                }\r
+            }\r
+        }\r
+        return result.toString();\r
+    }\r
+    \r
+    /**\r
+     * Return <code>true</code> if the given <code>str</code> matches at least one of the given <code>patterns</code>.\r
+     */\r
+    protected static boolean matchesOne(String str, Pattern... patterns) {\r
+        for (Pattern pattern : patterns) {\r
+            if (pattern.matcher(str).matches()) {\r
+                return true;\r
+            }\r
+        }\r
+        return false;\r
+    }\r
+    \r
+    /**\r
+     * @see #setHttpsServerPort(int)\r
+     */\r
+    private int httpsServerPort = 443;\r
+    \r
+    /**\r
+     * @see #setInternalProxies(String)\r
+     */\r
+    private Pattern[] internalProxies = new Pattern[] {\r
+        Pattern.compile("10\\.\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}"), Pattern.compile("192\\.168\\.\\d{1,3}\\.\\d{1,3}"),\r
+        Pattern.compile("169\\.254\\.\\d{1,3}\\.\\d{1,3}"), Pattern.compile("127\\.\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}")\r
+    };\r
+    \r
+    /**\r
+     * @see #setProtocolHeader(String)\r
+     */\r
+    private String protocolHeader = null;\r
+    \r
+    /**\r
+     * @see #setProtocolHeaderHttpsValue(String)\r
+     */\r
+    private String protocolHeaderHttpsValue = "https";\r
+    \r
+    /**\r
+     * @see #setProxiesHeader(String)\r
+     */\r
+    private String proxiesHeader = "X-Forwarded-By";\r
+    \r
+    /**\r
+     * @see #setRemoteIpHeader(String)\r
+     */\r
+    private String remoteIpHeader = "X-Forwarded-For";\r
+    \r
+    /**\r
+     * @see RemoteIpValve#setTrustedProxies(String)\r
+     */\r
+    private Pattern[] trustedProxies = new Pattern[0];\r
+    \r
+    public int getHttpsServerPort() {\r
+        return httpsServerPort;\r
+    }\r
+    \r
+    /**\r
+     * Return descriptive information about this Valve implementation.\r
+     */\r
+    @Override\r
+    public String getInfo() {\r
+        return info;\r
+    }\r
+    \r
+    /**\r
+     * @see #setInternalProxies(String)\r
+     * @return comma delimited list of internal proxies\r
+     */\r
+    public String getInternalProxies() {\r
+        List<String> internalProxiesAsStringList = new ArrayList<String>();\r
+        for (Pattern internalProxyPattern : internalProxies) {\r
+            internalProxiesAsStringList.add(String.valueOf(internalProxyPattern));\r
+        }\r
+        return listToCommaDelimitedString(internalProxiesAsStringList);\r
+    }\r
+    \r
+    /**\r
+     * @see #setProtocolHeader(String)\r
+     * @return the protocol header (e.g. "X-Forwarded-Proto")\r
+     */\r
+    public String getProtocolHeader() {\r
+        return protocolHeader;\r
+    }\r
+    \r
+    /**\r
+     * @see RemoteIpValve#setProtocolHeaderHttpsValue(String)\r
+     * @return the value of the protocol header for incoming https request (e.g. "https")\r
+     */\r
+    public String getProtocolHeaderHttpsValue() {\r
+        return protocolHeaderHttpsValue;\r
+    }\r
+    \r
+    /**\r
+     * @see #setProxiesHeader(String)\r
+     * @return the proxies header name (e.g. "X-Forwarded-By")\r
+     */\r
+    public String getProxiesHeader() {\r
+        return proxiesHeader;\r
+    }\r
+    \r
+    /**\r
+     * @see #setRemoteIpHeader(String)\r
+     * @return the remote IP header name (e.g. "X-Forwarded-For")\r
+     */\r
+    public String getRemoteIpHeader() {\r
+        return remoteIpHeader;\r
+    }\r
+    \r
+    /**\r
+     * @see #setTrustedProxies(String)\r
+     * @return comma delimited list of trusted proxies\r
+     */\r
+    public String getTrustedProxies() {\r
+        List<String> trustedProxiesAsStringList = new ArrayList<String>();\r
+        for (Pattern trustedProxy : trustedProxies) {\r
+            trustedProxiesAsStringList.add(String.valueOf(trustedProxy));\r
+        }\r
+        return listToCommaDelimitedString(trustedProxiesAsStringList);\r
+    }\r
+    \r
+    /**\r
+     * {@inheritDoc}\r
+     */\r
+    @Override\r
+    public void invoke(Request request, Response response) throws IOException, ServletException {\r
+        final String originalRemoteAddr = request.getRemoteAddr();\r
+        final String originalRemoteHost = request.getRemoteHost();\r
+        final String originalScheme = request.getScheme();\r
+        final boolean originalSecure = request.isSecure();\r
+        final int originalServerPort = request.getServerPort();\r
+        \r
+        if (matchesOne(originalRemoteAddr, internalProxies)) {\r
+            String remoteIp = null;\r
+            // In java 6, proxiesHeaderValue should be declared as a java.util.Deque\r
+            LinkedList<String> proxiesHeaderValue = new LinkedList<String>();\r
+            \r
+            String[] remoteIPHeaderValue = commaDelimitedListToStringArray(request.getHeader(remoteIpHeader));\r
+            int idx;\r
+            // loop on remoteIPHeaderValue to find the first trusted remote ip and to build the proxies chain\r
+            for (idx = remoteIPHeaderValue.length - 1; idx >= 0; idx--) {\r
+                String currentRemoteIp = remoteIPHeaderValue[idx];\r
+                remoteIp = currentRemoteIp;\r
+                if (matchesOne(currentRemoteIp, internalProxies)) {\r
+                    // do nothing, internalProxies IPs are not appended to the\r
+                } else if (matchesOne(currentRemoteIp, trustedProxies)) {\r
+                    proxiesHeaderValue.addFirst(currentRemoteIp);\r
+                } else {\r
+                    idx--; // decrement idx because break statement doesn't do it\r
+                    break;\r
+                }\r
+            }\r
+            // continue to loop on remoteIPHeaderValue to build the new value of the remoteIPHeader\r
+            LinkedList<String> newRemoteIpHeaderValue = new LinkedList<String>();\r
+            for (; idx >= 0; idx--) {\r
+                String currentRemoteIp = remoteIPHeaderValue[idx];\r
+                newRemoteIpHeaderValue.addFirst(currentRemoteIp);\r
+            }\r
+            if (remoteIp != null) {\r
+                \r
+                request.setRemoteAddr(remoteIp);\r
+                request.setRemoteHost(remoteIp);\r
+                \r
+                // use request.coyoteRequest.mimeHeaders.setValue(str).setString(str) because request.addHeader(str, str) is no-op in Tomcat\r
+                // 6.0\r
+                if (proxiesHeaderValue.size() == 0) {\r
+                    request.getCoyoteRequest().getMimeHeaders().removeHeader(proxiesHeader);\r
+                } else {\r
+                    String commaDelimitedListOfProxies = listToCommaDelimitedString(proxiesHeaderValue);\r
+                    request.getCoyoteRequest().getMimeHeaders().setValue(proxiesHeader).setString(commaDelimitedListOfProxies);\r
+                }\r
+                if (newRemoteIpHeaderValue.size() == 0) {\r
+                    request.getCoyoteRequest().getMimeHeaders().removeHeader(remoteIpHeader);\r
+                } else {\r
+                    String commaDelimitedRemoteIpHeaderValue = listToCommaDelimitedString(newRemoteIpHeaderValue);\r
+                    request.getCoyoteRequest().getMimeHeaders().setValue(remoteIpHeader).setString(commaDelimitedRemoteIpHeaderValue);\r
+                }\r
+            }\r
+            \r
+            if (protocolHeader != null) {\r
+                String protocolHeaderValue = request.getHeader(protocolHeader);\r
+                if (protocolHeaderValue != null && protocolHeaderHttpsValue.equalsIgnoreCase(protocolHeaderValue)) {\r
+                    request.setSecure(true);\r
+                    // use request.coyoteRequest.scheme instead of request.setScheme() because request.setScheme() is no-op in Tomcat 6.0\r
+                    request.getCoyoteRequest().scheme().setString("https");\r
+                    \r
+                    request.setServerPort(httpsServerPort);\r
+                }\r
+            }\r
+            \r
+            if (log.isDebugEnabled()) {\r
+                log.debug("Incoming request " + request.getRequestURI() + " with originalRemoteAddr '" + originalRemoteAddr\r
+                          + "', originalRemoteHost='" + originalRemoteHost + "', originalSecure='" + originalSecure + "', originalScheme='"\r
+                          + originalScheme + "' will be seen as newRemoteAddr='" + request.getRemoteAddr() + "', newRemoteHost='"\r
+                          + request.getRemoteHost() + "', newScheme='" + request.getScheme() + "', newSecure='" + request.isSecure() + "'");\r
+            }\r
+        }\r
+        try {\r
+            getNext().invoke(request, response);\r
+        } finally {\r
+            request.setRemoteAddr(originalRemoteAddr);\r
+            request.setRemoteHost(originalRemoteHost);\r
+            \r
+            request.setSecure(originalSecure);\r
+            \r
+            // use request.coyoteRequest.scheme instead of request.setScheme() because request.setScheme() is no-op in Tomcat 6.0\r
+            request.getCoyoteRequest().scheme().setString(originalScheme);\r
+            \r
+            request.setServerPort(originalServerPort);\r
+        }\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * Server Port value if the {@link #protocolHeader} indicates HTTPS\r
+     * </p>\r
+     * <p>\r
+     * Default value : 443\r
+     * </p>\r
+     */\r
+    public void setHttpsServerPort(int httpsServerPort) {\r
+        this.httpsServerPort = httpsServerPort;\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * Comma delimited list of internal proxies. Can be expressed with regular expressions.\r
+     * </p>\r
+     * <p>\r
+     * Default value : 10\.\d{1,3}\.\d{1,3}\.\d{1,3}, 192\.168\.\d{1,3}\.\d{1,3}, 127\.\d{1,3}\.\d{1,3}\.\d{1,3}\r
+     * </p>\r
+     */\r
+    public void setInternalProxies(String commaDelimitedInternalProxies) {\r
+        this.internalProxies = commaDelimitedListToPatternArray(commaDelimitedInternalProxies);\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * Header that holds the incoming protocol, usally named <code>X-Forwarded-Proto</code>. If <code>null</code>, request.scheme and\r
+     * request.secure will not be modified.\r
+     * </p>\r
+     * <p>\r
+     * Default value : <code>null</code>\r
+     * </p>\r
+     */\r
+    public void setProtocolHeader(String protocolHeader) {\r
+        this.protocolHeader = protocolHeader;\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * Case insensitive value of the protocol header to indicate that the incoming http request uses SSL.\r
+     * </p>\r
+     * <p>\r
+     * Default value : <code>https</code>\r
+     * </p>\r
+     */\r
+    public void setProtocolHeaderHttpsValue(String protocolHeaderHttpsValue) {\r
+        this.protocolHeaderHttpsValue = protocolHeaderHttpsValue;\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * The proxiesHeader directive specifies a header into which mod_remoteip will collect a list of all of the intermediate client IP\r
+     * addresses trusted to resolve the actual remote IP. Note that intermediate RemoteIPTrustedProxy addresses are recorded in this header,\r
+     * while any intermediate RemoteIPInternalProxy addresses are discarded.\r
+     * </p>\r
+     * <p>\r
+     * Name of the http header that holds the list of trusted proxies that has been traversed by the http request.\r
+     * </p>\r
+     * <p>\r
+     * The value of this header can be comma delimited.\r
+     * </p>\r
+     * <p>\r
+     * Default value : <code>X-Forwarded-By</code>\r
+     * </p>\r
+     */\r
+    public void setProxiesHeader(String proxiesHeader) {\r
+        this.proxiesHeader = proxiesHeader;\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * Name of the http header from which the remote ip is extracted.\r
+     * </p>\r
+     * <p>\r
+     * The value of this header can be comma delimited.\r
+     * </p>\r
+     * <p>\r
+     * Default value : <code>X-Forwarded-For</code>\r
+     * </p>\r
+     * \r
+     * @param remoteIPHeader\r
+     */\r
+    public void setRemoteIpHeader(String remoteIpHeader) {\r
+        this.remoteIpHeader = remoteIpHeader;\r
+    }\r
+    \r
+    /**\r
+     * <p>\r
+     * Comma delimited list of proxies that are trusted when they appear in the {@link #remoteIPHeader} header. Can be expressed as a\r
+     * regular expression.\r
+     * </p>\r
+     * <p>\r
+     * Default value : empty list, no external proxy is trusted.\r
+     * </p>\r
+     */\r
+    public void setTrustedProxies(String commaDelimitedTrustedProxies) {\r
+        this.trustedProxies = commaDelimitedListToPatternArray(commaDelimitedTrustedProxies);\r
+    }\r
+}\r
index 3448c9a..76ecf30 100644 (file)
 
   </mbean>
 
+  <mbean name="RemoteIpValve"
+         description="Valve that sets client information (eg IP address) based on data from a trusted proxy"
+         domain="Catalina"
+         group="Valve"
+         type="org.apache.catalina.valves.RemoteIpValve">
+    
+    <attribute name="internalProxies"
+               description="Comma delimited list of internal proxies"
+               type="java.lang.String"
+               writeable="false" />
+               
+    <attribute name="protocolHeader"
+               description="The protocol header (e.g. &quot;X-Forwarded-Proto&quot;)"
+               type="java.lang.String"
+               writeable="false" />
+               
+    <attribute name="protocolHeaderHttpsValue"
+               description="The value of the protocol header for incoming https request (e.g. &quot;https&quot;)"
+               type="java.lang.String"
+               writeable="false" />
+               
+    <attribute name="proxiesHeader"
+               description="The proxies header name (e.g. &quot;X-Forwarded-By&quot;)"
+               type="java.lang.String"
+               writeable="false" />
+               
+    <attribute name="remoteIpHedaer"
+               description="The remote IP header name (e.g. &quot;X-Forwarded-For&quot;)"
+               type="java.lang.String"
+               writeable="false" />
+               
+    <attribute name="trustedProxies"
+               description="Comma delimited list of trusted proxies"
+               type="java.lang.String"
+               writeable="false" />
+               
+  </mbean>
 </mbeans-descriptors>
diff --git a/test/org/apache/catalina/valves/RemoteIpValveTest.java b/test/org/apache/catalina/valves/RemoteIpValveTest.java
new file mode 100644 (file)
index 0000000..3afee68
--- /dev/null
@@ -0,0 +1,386 @@
+/*\r
+ * Licensed to the Apache Software Foundation (ASF) under one or more\r
+ * contributor license agreements.  See the NOTICE file distributed with\r
+ * this work for additional information regarding copyright ownership.\r
+ * The ASF licenses this file to You under the Apache License, Version 2.0\r
+ * (the "License"); you may not use this file except in compliance with\r
+ * the License.  You may obtain a copy of the License at\r
+ * \r
+ *      http://www.apache.org/licenses/LICENSE-2.0\r
+ * \r
+ * Unless required by applicable law or agreed to in writing, software\r
+ * distributed under the License is distributed on an "AS IS" BASIS,\r
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
+ * See the License for the specific language governing permissions and\r
+ * limitations under the License.\r
+ */\r
+\r
+package org.apache.catalina.valves;\r
+\r
+import java.io.IOException;\r
+import java.util.ArrayList;\r
+import java.util.Arrays;\r
+import java.util.List;\r
+\r
+import javax.servlet.ServletException;\r
+\r
+import junit.framework.TestCase;\r
+\r
+import org.apache.catalina.connector.Request;\r
+import org.apache.catalina.connector.Response;\r
+import org.apache.catalina.valves.ValveBase;\r
+\r
+/**\r
+ * {@link RemoteIpValve} Tests\r
+ */\r
+public class RemoteIpValveTest extends TestCase {\r
+    \r
+    static class RemoteAddrAndHostTrackerValve extends ValveBase {\r
+        private String remoteAddr;\r
+        private String remoteHost;\r
+        \r
+        public String getRemoteAddr() {\r
+            return remoteAddr;\r
+        }\r
+        \r
+        public String getRemoteHost() {\r
+            return remoteHost;\r
+        }\r
+        \r
+        @Override\r
+        public void invoke(Request request, Response response) throws IOException, ServletException {\r
+            this.remoteHost = request.getRemoteHost();\r
+            this.remoteAddr = request.getRemoteAddr();\r
+        }\r
+    }\r
+    \r
+    public void testCommaDelimitedListToStringArray() {\r
+        List<String> elements = Arrays.asList("element1", "element2", "element3");\r
+        String actual = RemoteIpValve.listToCommaDelimitedString(elements);\r
+        assertEquals("element1, element2, element3", actual);\r
+    }\r
+    \r
+    public void testCommaDelimitedListToStringArrayEmptyList() {\r
+        List<String> elements = new ArrayList<String>();\r
+        String actual = RemoteIpValve.listToCommaDelimitedString(elements);\r
+        assertEquals("", actual);\r
+    }\r
+    \r
+    public void testCommaDelimitedListToStringArrayNullList() {\r
+        String actual = RemoteIpValve.listToCommaDelimitedString(null);\r
+        assertEquals("", actual);\r
+    }\r
+    \r
+    public void testInvokeAllowedRemoteAddrWithNullRemoteIpHeader() throws Exception {\r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("192\\.168\\.0\\.10, 192\\.168\\.0\\.11");\r
+        remoteIpValve.setTrustedProxies("proxy1, proxy2, proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("192.168.0.10");\r
+        request.setRemoteHost("remote-host-original-value");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertNull("x-forwarded-for must be null", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertNull("x-forwarded-by must be null", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "192.168.0.10", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "remote-host-original-value", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "192.168.0.10", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "remote-host-original-value", actualPostInvokeRemoteHost);\r
+        \r
+    }\r
+    \r
+    public void testInvokeAllProxiesAreTrusted() throws Exception {\r
+        \r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("192\\.168\\.0\\.10, 192\\.168\\.0\\.11");\r
+        remoteIpValve.setTrustedProxies("proxy1, proxy2, proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("192.168.0.10");\r
+        request.setRemoteHost("remote-host-original-value");\r
+        request.getCoyoteRequest().getMimeHeaders().addValue("x-forwarded-for").setString("140.211.11.130, proxy1, proxy2");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertNull("all proxies are trusted, x-forwarded-for must be null", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertEquals("all proxies are trusted, they must appear in x-forwarded-by", "proxy1, proxy2", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "140.211.11.130", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "140.211.11.130", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "192.168.0.10", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "remote-host-original-value", actualPostInvokeRemoteHost);\r
+    }\r
+    \r
+    public void testInvokeAllProxiesAreTrustedOrInternal() throws Exception {\r
+        \r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("192\\.168\\.0\\.10, 192\\.168\\.0\\.11");\r
+        remoteIpValve.setTrustedProxies("proxy1, proxy2, proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("192.168.0.10");\r
+        request.setRemoteHost("remote-host-original-value");\r
+        request.getCoyoteRequest().getMimeHeaders().addValue("x-forwarded-for")\r
+            .setString("140.211.11.130, proxy1, proxy2, 192.168.0.10, 192.168.0.11");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertNull("all proxies are trusted, x-forwarded-for must be null", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertEquals("all proxies are trusted, they must appear in x-forwarded-by", "proxy1, proxy2", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "140.211.11.130", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "140.211.11.130", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "192.168.0.10", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "remote-host-original-value", actualPostInvokeRemoteHost);\r
+    }\r
+    \r
+    public void testInvokeAllProxiesAreInternal() throws Exception {\r
+        \r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("192\\.168\\.0\\.10, 192\\.168\\.0\\.11");\r
+        remoteIpValve.setTrustedProxies("proxy1, proxy2, proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("192.168.0.10");\r
+        request.setRemoteHost("remote-host-original-value");\r
+        request.getCoyoteRequest().getMimeHeaders().addValue("x-forwarded-for").setString("140.211.11.130, 192.168.0.10, 192.168.0.11");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertNull("all proxies are internal, x-forwarded-for must be null", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertNull("all proxies are internal, x-forwarded-by must be null", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "140.211.11.130", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "140.211.11.130", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "192.168.0.10", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "remote-host-original-value", actualPostInvokeRemoteHost);\r
+    }\r
+    \r
+    public void testInvokeAllProxiesAreTrustedAndRemoteAddrMatchRegexp() throws Exception {\r
+        \r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("127\\.0\\.0\\.1, 192\\.168\\..*, another-internal-proxy");\r
+        remoteIpValve.setTrustedProxies("proxy1, proxy2, proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("192.168.0.10");\r
+        request.setRemoteHost("remote-host-original-value");\r
+        request.getCoyoteRequest().getMimeHeaders().addValue("x-forwarded-for").setString("140.211.11.130, proxy1, proxy2");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertNull("all proxies are trusted, x-forwarded-for must be null", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertEquals("all proxies are trusted, they must appear in x-forwarded-by", "proxy1, proxy2", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "140.211.11.130", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "140.211.11.130", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "192.168.0.10", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "remote-host-original-value", actualPostInvokeRemoteHost);\r
+    }\r
+    \r
+    public void testInvokeNotAllowedRemoteAddr() throws Exception {\r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("192\\.168\\.0\\.10, 192\\.168\\.0\\.11");\r
+        remoteIpValve.setTrustedProxies("proxy1,proxy2,proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("not-allowed-internal-proxy");\r
+        request.setRemoteHost("not-allowed-internal-proxy-host");\r
+        request.getCoyoteRequest().getMimeHeaders().addValue("x-forwarded-for").setString("140.211.11.130, proxy1, proxy2");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertEquals("x-forwarded-for must be unchanged", "140.211.11.130, proxy1, proxy2", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertNull("x-forwarded-by must be null", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "not-allowed-internal-proxy", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "not-allowed-internal-proxy-host", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "not-allowed-internal-proxy", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "not-allowed-internal-proxy-host", actualPostInvokeRemoteHost);\r
+    }\r
+    \r
+    public void testInvokeUntrustedProxyInTheChain() throws Exception {\r
+        // PREPARE\r
+        RemoteIpValve remoteIpValve = new RemoteIpValve();\r
+        remoteIpValve.setInternalProxies("192\\.168\\.0\\.10, 192\\.168\\.0\\.11");\r
+        remoteIpValve.setTrustedProxies("proxy1, proxy2, proxy3");\r
+        remoteIpValve.setRemoteIpHeader("x-forwarded-for");\r
+        remoteIpValve.setProxiesHeader("x-forwarded-by");\r
+        RemoteAddrAndHostTrackerValve remoteAddrAndHostTrackerValve = new RemoteAddrAndHostTrackerValve();\r
+        remoteIpValve.setNext(remoteAddrAndHostTrackerValve);\r
+        \r
+        Request request = new Request();\r
+        request.setCoyoteRequest(new org.apache.coyote.Request());\r
+        request.setRemoteAddr("192.168.0.10");\r
+        request.setRemoteHost("remote-host-original-value");\r
+        request.getCoyoteRequest().getMimeHeaders().addValue("x-forwarded-for")\r
+            .setString("140.211.11.130, proxy1, untrusted-proxy, proxy2");\r
+        \r
+        // TEST\r
+        remoteIpValve.invoke(request, null);\r
+        \r
+        // VERIFY\r
+        String actualXForwardedFor = request.getHeader("x-forwarded-for");\r
+        assertEquals("ip/host before untrusted-proxy must appear in x-forwarded-for", "140.211.11.130, proxy1", actualXForwardedFor);\r
+        \r
+        String actualXForwardedBy = request.getHeader("x-forwarded-by");\r
+        assertEquals("ip/host after untrusted-proxy must appear in  x-forwarded-by", "proxy2", actualXForwardedBy);\r
+        \r
+        String actualRemoteAddr = remoteAddrAndHostTrackerValve.getRemoteAddr();\r
+        assertEquals("remoteAddr", "untrusted-proxy", actualRemoteAddr);\r
+        \r
+        String actualRemoteHost = remoteAddrAndHostTrackerValve.getRemoteHost();\r
+        assertEquals("remoteHost", "untrusted-proxy", actualRemoteHost);\r
+        \r
+        String actualPostInvokeRemoteAddr = request.getRemoteAddr();\r
+        assertEquals("postInvoke remoteAddr", "192.168.0.10", actualPostInvokeRemoteAddr);\r
+        \r
+        String actualPostInvokeRemoteHost = request.getRemoteHost();\r
+        assertEquals("postInvoke remoteAddr", "remote-host-original-value", actualPostInvokeRemoteHost);\r
+    }\r
+    \r
+    public void testListToCommaDelimitedString() {\r
+        String[] actual = RemoteIpValve.commaDelimitedListToStringArray("element1, element2, element3");\r
+        String[] expected = new String[] {\r
+            "element1", "element2", "element3"\r
+        };\r
+        assertArrayEquals(expected, actual);\r
+    }\r
+    \r
+    public void testListToCommaDelimitedStringMixedSpaceChars() {\r
+        String[] actual = RemoteIpValve.commaDelimitedListToStringArray("element1  , element2,\t element3");\r
+        String[] expected = new String[] {\r
+            "element1", "element2", "element3"\r
+        };\r
+        assertArrayEquals(expected, actual);\r
+    }\r
+    \r
+    private void assertArrayEquals(String[] expected, String[] actual) {\r
+        if (expected == null) {\r
+            assertNull(actual);\r
+            return;\r
+        }\r
+        assertNotNull(actual);\r
+        assertEquals(expected.length, actual.length);\r
+        List<String> e = new ArrayList<String>();\r
+        e.addAll(Arrays.asList(expected));\r
+        List<String> a = new ArrayList<String>();\r
+        a.addAll(Arrays.asList(actual));\r
+        \r
+        for (String entry : e) {\r
+            assertTrue(a.remove(entry));\r
+        }\r
+        assertTrue(a.isEmpty());\r
+    }\r
+}\r
index b8ecf69..291c42e 100644 (file)
 </section>
 
 
+<section name="Remote IP Valve">
+
+  <subsection name="Introduction">
+  
+    <p>Tomcat port of
+    <a href="http://httpd.apache.org/docs/trunk/mod/mod_remoteip.html">mod_remoteip</a>,
+    this valve replaces the apparent client remote IP address and hostname for
+    the request with the IP address list presented by a proxy or a load balancer
+    via a request headers (e.g. &quot;X-Forwarded-For&quot;).</p>
+
+    <p>Another feature of this valve is to replace the apparent scheme
+    (http/https) and server port with the scheme presented by a proxy or a load
+    balancer via a request header (e.g. &quot;X-Forwarded-Proto&quot;).</p>
+    <p>This Valve may be used at the <code>Engine</code>, <code>Host</code> or
+    <code>Context</code> level as required. Normally, this Valve would be used
+    at the <code>Engine</code> level.</p>
+    
+    <p>If used in conjunction with Remote Address/Host valves then this valve
+    should be defined first to ensure that the correct client IP address is
+    presented to the Remote Address/Host valves.</p>
+
+  </subsection>
+
+  <subsection name="Attributes">
+
+    <p>The <strong>Remote IP Valve</strong> supports the
+    following configuration attributes:</p>
+
+    <attributes>
+
+      <attribute name="className" required="true">
+        <p>Java class name of the implementation to use.  This MUST be set to
+        <strong>org.apache.catalina.valves.RemoteIpValve</strong>.</p>
+      </attribute>
+
+      <attribute name="remoteIPHeader" required="false">
+        <p>Name of the HTTP Header read by this valve that holds the list of
+        traversed IP addresses starting from the requesting client. If not
+        specified, the default of <code>x-forwarded-for</code> is used.</p>
+      </attribute>
+
+      <attribute name="internalProxies" required="false">
+        <p>List of internal proxies' IP addresses as comma separated regular
+        expressions. If they appear in the <strong>remoteIpHeader</strong>
+        value, they will be trusted and will not appear in the
+        <strong>proxiesHeader</strong> value. If not specified the default value
+        of <code>10\.\d{1,3}\.\d{1,3}\.\d{1,3}, 192\.168\.\d{1,3}\.\d{1,3},
+        169\.254\.\d{1,3}\.\d{1,3}, 127\.\d{1,3}\.\d{1,3}\.\d{1,3}</code> will
+        be used.</p>
+      </attribute>
+
+      <attribute name="proxiesHeader" required="false">
+        <p>Name of the HTTP header created by this valve to hold the list of
+        proxies that have been processed in the incoming
+        <strong>remoteIpHeader</strong>. If not specified, the default of
+        <code>x-forwarded-by</code> is used.</p>
+      </attribute>
+
+      <attribute name="trustedProxies" required="false">
+        <p>List of trusted proxies' IP addresses as comma separated regular
+        expressions. If they appear in the <strong>remoteIpHeader</strong>
+        value, they will be trusted and will appear in the
+        <strong>proxiesHeader</strong> value. If not specified, no proxies will
+        be trusted.</p>
+      </attribute>
+
+      <attribute name="protocolHeader" required="false">
+        <p>Name of the HTTP Header read by this valve that holds the protocol
+        used by the client to connect to the proxy. If not specified, the
+        default of <code>null</code> is used.</p>
+      </attribute>
+
+      <attribute name="protocolHeaderHttpsValue" required="false">
+        <p>Value of the <strong>protocolHeader</strong> to indicate that it is
+        an HTTPS request. If not specified, the default of <code>https</code> is
+        used.</p>
+      </attribute>
+
+    </attributes>
+
+  </subsection>
+
+</section>
+
+
 </body>