summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorCarl Schwan <carl@carlschwan.eu>2022-06-29 17:05:48 +0200
committerCarl Schwan <carl@carlschwan.eu>2022-06-29 17:05:48 +0200
commitca3cd5a6257b4e40a72ac9327989deb2801015a0 (patch)
tree9ea68ad3a28890662b2a8e55049209c6b11eeb59 /lib
parent1952d6e9b4d9a955ba2f93e797541833bcc6b18a (diff)
downloadnextcloud-server-ca3cd5a6257b4e40a72ac9327989deb2801015a0.tar.gz
nextcloud-server-ca3cd5a6257b4e40a72ac9327989deb2801015a0.zip
Fix detection of firefox in ContentSecurityPolicyNonceManager
Reuse Request::USER_AGENT_FIREFOX, and also update the safari detection since safari < 12 is not supported anymore and we can remove a bit of code duplication Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Diffstat (limited to 'lib')
-rw-r--r--lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php6
1 files changed, 2 insertions, 4 deletions
diff --git a/lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php b/lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php
index f3329135727..1167b3358d2 100644
--- a/lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php
+++ b/lib/private/Security/CSP/ContentSecurityPolicyNonceManager.php
@@ -80,10 +80,8 @@ class ContentSecurityPolicyNonceManager {
public function browserSupportsCspV3(): bool {
$browserWhitelist = [
Request::USER_AGENT_CHROME,
- // Firefox 45+
- '/^Mozilla\/5\.0 \([^)]+\) Gecko\/[0-9.]+ Firefox\/(4[5-9]|[5-9][0-9])\.[0-9.]+$/',
- // Safari 12+
- '/^Mozilla\/5\.0 \([^)]+\) AppleWebKit\/[0-9.]+ \(KHTML, like Gecko\) Version\/(?:1[2-9]|[2-9][0-9])\.[0-9]+(?:\.[0-9]+)? Safari\/[0-9.A-Z]+$/',
+ Request::USER_AGENT_FIREFOX,
+ Request::USER_AGENT_SAFARI,
];
if ($this->request->isUserAgent($browserWhitelist)) {