[ 
https://issues.apache.org/jira/browse/WICKET-7172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18049097#comment-18049097
 ] 

ASF GitHub Bot commented on WICKET-7172:
----------------------------------------

martin-g commented on code in PR #1341:
URL: https://github.com/apache/wicket/pull/1341#discussion_r2660699818


##########
wicket-core/src/main/java/org/apache/wicket/csp/CSPDirective.java:
##########
@@ -147,6 +151,19 @@ public String getValue()
        public void checkValueForDirective(CSPRenderable value,
                        List<CSPRenderable> existingDirectiveValues)
        {
+               if (this == SCRIPT_SRC_ATTR || this == STYLE_SRC_ATTR) 
+               {
+                       if (!existingDirectiveValues.isEmpty()) 
+                       {
+                               throw new IllegalArgumentException("Directive " 
+ this + " supports only one value");

Review Comment:
   According to 
https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/Content-Security-Policy/script-src-attr#source-expression-list
 those could have multiple values, not just one.





> Support new CSP style, script directives
> ----------------------------------------
>
>                 Key: WICKET-7172
>                 URL: https://issues.apache.org/jira/browse/WICKET-7172
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket-core
>            Reporter: Kees van Dieren
>            Priority: Major
>
> Support new CSP directives added to the CSP in 2022
> They where not yet supported by Wicket.
> See: https://github.com/apache/wicket/pull/1341
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to