michael-o commented on code in PR #681:
URL: https://github.com/apache/tomcat/pull/681#discussion_r1430459964


##########
java/org/apache/catalina/filters/CsrfPreventionFilter.java:
##########
@@ -53,6 +58,25 @@ public class CsrfPreventionFilter extends 
CsrfPreventionFilterBase {
 
     private String nonceRequestParameterName = 
Constants.CSRF_NONCE_REQUEST_PARAM;
 
+    private boolean enforce = true;
+
+    private Collection<Predicate<String>> noNoncePatterns = 
DEFAULT_NO_NONCE_URL_PATTERNS;
+
+    private static final Collection<Predicate<String>> 
DEFAULT_NO_NONCE_URL_PATTERNS;
+
+    static {
+        ArrayList<Predicate<String>> defaultNoNonceURLPatterns = new 
ArrayList<>();
+
+        defaultNoNonceURLPatterns.add(new SuffixPredicate(".css"));
+        defaultNoNonceURLPatterns.add(new SuffixPredicate(".js"));
+        defaultNoNonceURLPatterns.add(new SuffixPredicate(".gif"));
+        defaultNoNonceURLPatterns.add(new SuffixPredicate(".png"));
+        defaultNoNonceURLPatterns.add(new SuffixPredicate(".jpg"));

Review Comment:
   The original extension is `.jpeg`. `.jpg` only exists because IBM's Disk 
Operating System as too stupid to handle more than three chars for file 
extensions.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to