Browse Source

fix(CSP): Add CSP nonce by default and convert `browserSupportsCspV3` to blocklist

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Ferdinand Thiessen 1 month ago
parent
commit
5a513c924f
1 changed files with 5 additions and 8 deletions
  1. 5 8
      lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php

+ 5 - 8
lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php

@@ -65,17 +65,14 @@ class ContentSecurityPolicyNonceManager {
 	 * Check if the browser supports CSP v3
 	 */
 	public function browserSupportsCspV3(): bool {
-		$browserWhitelist = [
-			Request::USER_AGENT_CHROME,
-			Request::USER_AGENT_FIREFOX,
-			Request::USER_AGENT_SAFARI,
-			Request::USER_AGENT_MS_EDGE,
+		$browserBlocklist = [
+			Request::USER_AGENT_IE,
 		];
 
-		if ($this->request->isUserAgent($browserWhitelist)) {
-			return true;
+		if ($this->request->isUserAgent($browserBlocklist)) {
+			return false;
 		}
 
-		return false;
+		return true;
 	}
 }