add comments to help explain usages of the various CSP declarations (#…

…991) * add comments to help explain usages of the various CSP declarations -- Note: comments are removed during application deployment
LabKey · Feb 19, 2025 · 3909ce7 · 3909ce7
1 parent 686918f
commit 3909ce7
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/server/configs/application.properties b/server/configs/application.properties
@@ -141,16 +141,16 @@ management.server.port=@@shutdownPort@@
 
 ## START OF CSP REPORT BLOCK (DO NOT CHANGE THIS TEXT)
 csp.report=\
-    default-src 'self' ;\
-    connect-src 'self' ${LABKEY.ALLOWED.CONNECTIONS} ;\
-    object-src 'none' ;\
-    style-src 'self' 'unsafe-inline' ;\
-    img-src 'self' data: ;\
-    font-src 'self' data: ;\
-    script-src 'unsafe-eval' 'strict-dynamic' 'nonce-${REQUEST.SCRIPT.NONCE}' ;\
-    base-uri 'self' ;\
-    frame-ancestors 'self' ;\
-    report-uri /admin-contentsecuritypolicyreport.api?${CSP.REPORT.PARAMS} ;
+    default-src 'self' ;                                                            /* Limit the default to only the current server */\
+    connect-src 'self' ${LABKEY.ALLOWED.CONNECTIONS} ;                              /* For security purposes limit allowed connection sources, can be substituted and appended via the LabKey Admin UI */\
+    object-src 'none' ;                                                             /* These tags are not currently used by LKS */\
+    style-src 'self' 'unsafe-inline' ;                                              /* We currently have a few inline <style> tags that we are weeding out */\
+    img-src 'self' data: ;                                                          /* Limit image loading locations */\
+    font-src 'self' data: ;                                                         /* Limit font source loading locations */\
+    script-src 'unsafe-eval' 'strict-dynamic' 'nonce-${REQUEST.SCRIPT.NONCE}' ;     /* Limit scripts that are allowed to those with nonces or transitive scripts */\
+    base-uri 'self' ;                                                               /* Limit the base tags to only source from current server */\
+    frame-ancestors 'self' ;                                                        /* Only allow embedding resources to the current server */\
+    report-uri /admin-contentsecuritypolicyreport.api?${CSP.REPORT.PARAMS} ;        /* Reports any encountered CSP conflicts to the supplied URL */
 ## END OF CSP REPORT BLOCK (DO NOT CHANGE THIS TEXT)
 
 ## Use a custom logging configuration