Update java/ql/src/experimental/Security/CWE/CWE-348/UseOfLessTrustedSource.qhelp

haby0 · smowton · web-flow · commit 9ece4dac0ff0 · 2021-04-20T19:33:47.000+08:00
Co-authored-by: Chris Smowton &lt;smowton@github.com&gt;
diff --git a/java/ql/src/experimental/Security/CWE/CWE-348/UseOfLessTrustedSource.qhelp b/java/ql/src/experimental/Security/CWE/CWE-348/UseOfLessTrustedSource.qhelp
@@ -10,7 +10,7 @@ bypass a ban-list, for example.</p>
 </overview>
 <recommendation>
 
-<p>When the software is not using a proxy server, get the last ip.</p>
+<p>Do not trust the values of HTTP headers allegedly identifying the originating IP. If you are aware your application will run behind some reverse proxies then the last entry of a <code>X-Forwarded-For</code> header value may be more trustworthy than the rest of it because some reverse proxies append the IP address they observed to the end of any remote-supplied header.</p>
 
 </recommendation>
 <example>
