diff --git a/CHANGELOG.md b/CHANGELOG.md
index 34d9485a..9ed1e3a3 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -3,6 +3,7 @@
## 1.7.4 (not yet released)
* CHANGED: Saving markdown pastes uses `.md` extension instead of `.txt` (#1293)
* CHANGED: Enable strict type checking in PHP (#1350)
+* CHANGED: SRI hashes are now configurable, no longer hardcoded in templates (#1365)
* FIXED: Reset password input field on creation of new paste (#1194)
* FIXED: Allow database schema upgrade to skip versions (#1343)
* FIXED: `bootstrap5` dark mode toggle unset on dark browser preference (#1340)
diff --git a/cfg/conf.sample.php b/cfg/conf.sample.php
index f1db2644..f91c5e89 100644
--- a/cfg/conf.sample.php
+++ b/cfg/conf.sample.php
@@ -275,3 +275,9 @@ dir = PATH "data"
; signature = ""
; (optional) the URL of the YOURLS API, called to shorten a PrivateBin URL
; apiurl = "https://yourls.example.com/yourls-api.php"
+
+;[sri]
+; Subresource integrity (SRI) hashes used in template files. Uncomment and set
+; these for all js files used. See:
+; https://github.com/PrivateBin/PrivateBin/wiki/FAQ#user-content-how-to-make-privatebin-work-when-i-have-changed-some-javascript-files
+;privatebin.js = sha512-[…]
diff --git a/lib/Configuration.php b/lib/Configuration.php
index 59722d15..e441f05f 100644
--- a/lib/Configuration.php
+++ b/lib/Configuration.php
@@ -98,6 +98,23 @@ class Configuration
'signature' => '',
'apiurl' => '',
),
+ // update this array when adding/changing/removing js files
+ 'sri' => array(
+ 'js/base-x-4.0.0.js' => 'sha512-nNPg5IGCwwrveZ8cA/yMGr5HiRS5Ps2H+s0J/mKTPjCPWUgFGGw7M5nqdnPD3VsRwCVysUh3Y8OWjeSKGkEQJQ==',
+ 'js/base64-1.7.js' => 'sha512-JdwsSP3GyHR+jaCkns9CL9NTt4JUJqm/BsODGmYhBcj5EAPKcHYh+OiMfyHbcDLECe17TL0hjXADFkusAqiYgA==',
+ 'js/bootstrap-3.4.1.js' => 'sha512-oBTprMeNEKCnqfuqKd6sbvFzmFQtlXS3e0C/RGFV0hD6QzhHV+ODfaQbAlmY6/q0ubbwlAM/nCJjkrgA3waLzg==',
+ 'js/bootstrap-5.3.3.js' => 'sha512-in2rcOpLTdJ7/pw5qjF4LWHFRtgoBDxXCy49H4YGOcVdGiPaQucGIbOqxt1JvmpvOpq3J/C7VTa0FlioakB2gQ==',
+ 'js/dark-mode-switch.js' => 'sha512-CCbdHdeWDbDO7aqFFmhgnvFESzaILHbUYmbhNjTpcjyO/XYdouQ9Pw8W9rpV8oJT1TsK5FbwSHU1oazmnb7BWA==',
+ 'js/jquery-3.7.1.js' => 'sha512-v2CJ7UaYy4JwqLDIrZUI/4hqeoQieOmAZNXBeQyjo21dadnwR+8ZaIJVT8EE2iyI61OV8e6M8PP2/4hpQINQ/g==',
+ 'js/kjua-0.9.0.js' => 'sha512-CVn7af+vTMBd9RjoS4QM5fpLFEOtBCoB0zPtaqIDC7sF4F8qgUSRFQQpIyEDGsr6yrjbuOLzdf20tkHHmpaqwQ==',
+ 'js/legacy.js' => 'sha512-LYos+qXHIRqFf5ZPNphvtTB0cgzHUizu2wwcOwcwz/VIpRv9lpcBgPYz4uq6jx0INwCAj6Fbnl5HoKiLufS2jg==',
+ 'js/prettify.js' => 'sha512-puO0Ogy++IoA2Pb9IjSxV1n4+kQkKXYAEUtVzfZpQepyDPyXk8hokiYDS7ybMogYlyyEIwMLpZqVhCkARQWLMg==',
+ 'js/privatebin.js' => 'sha512-cbmXvtZ/5gZPFjQDzP3IEhUAIhFPAoM31gw2kRYCT5xOh8wv9gXeDqI/t798luRW1xdC4gaYodjEFCzrsZR4mA==',
+ 'js/purify-3.1.3.js' => 'sha512-t/FKG/ucQVMWTWVouSMABSEx1r+uSyAI9eNDq0KEr9mPhkgxpJztHI/E72JIpv/+VwPs/Q4husxj14TE9Ps/wg==',
+ 'js/rawinflate-0.3.js' => 'sha512-g8uelGgJW9A/Z1tB6Izxab++oj5kdD7B4qC7DHwZkB6DGMXKyzx7v5mvap2HXueI2IIn08YlRYM56jwWdm2ucQ==',
+ 'js/showdown-2.1.0.js' => 'sha512-WYXZgkTR0u/Y9SVIA4nTTOih0kXMEd8RRV6MLFdL6YU8ymhR528NLlYQt1nlJQbYz4EW+ZsS0fx1awhiQJme1Q==',
+ 'js/zlib-1.3.1.js' => 'sha512-Z90oppVx/mn0DG2k9airjFVQuliELlXLeT3SRiO6MLiUSbhGlAq+UFwmYbG4i9mwW87dkG8fgJPapGwnUq7Osg==',
+ ),
);
/**
diff --git a/lib/Controller.php b/lib/Controller.php
index d518fcc5..20a83a13 100644
--- a/lib/Controller.php
+++ b/lib/Controller.php
@@ -436,6 +436,7 @@ class Controller
$page->assign('HTTPWARNING', $this->_conf->getKey('httpwarning'));
$page->assign('HTTPSLINK', 'https://' . $this->_request->getHost() . $this->_request->getRequestUri());
$page->assign('COMPRESSION', $this->_conf->getKey('compression'));
+ $page->assign('SRI', $this->_conf->getSection('sri'));
$page->draw($this->_conf->getKey('template'));
}
diff --git a/lib/View.php b/lib/View.php
index 395ac86d..8c21f341 100644
--- a/lib/View.php
+++ b/lib/View.php
@@ -57,4 +57,22 @@ class View
extract($this->_variables);
include $path;
}
+
+ /**
+ * echo script tag incl. SRI hash for given script file
+ *
+ * @access private
+ * @param string $file
+ * @param bool $async should it execute ASAP or only after HTML got parsed
+ */
+ private function _scriptTag($file, $async = true)
+ {
+ $sri = array_key_exists($file, $this->_variables['SRI']) ?
+ ' integrity="' . $this->_variables['SRI'][$file] . '"' : '';
+ $suffix = preg_match('#\d.js$#', $file) == 0 ?
+ '?' . rawurlencode($this->_variables['VERSION']) : '';
+ echo '', PHP_EOL;
+ }
}
diff --git a/tpl/bootstrap.php b/tpl/bootstrap.php
index e9a15ef1..2ae57582 100644
--- a/tpl/bootstrap.php
+++ b/tpl/bootstrap.php
@@ -42,38 +42,38 @@ if ($SYNTAXHIGHLIGHTING) :
endif;
?>
-
+ _scriptTag('js/jquery-3.7.1.js', false); ?>
-
+ _scriptTag('js/kjua-0.9.0.js'); ?>
-
+ _scriptTag('js/base64-1.7.js'); ?>
-
-
-
-
+ _scriptTag('js/zlib-1.3.1.js'); ?>
+ _scriptTag('js/base-x-4.0.0.js'); ?>
+ _scriptTag('js/rawinflate-0.3.js'); ?>
+ _scriptTag('js/bootstrap-3.4.1.js', false); ?>
-
+ _scriptTag('js/prettify.js'); ?>
-
+ _scriptTag('js/showdown-2.1.0.js'); ?>
-
-
-
+ _scriptTag('js/purify-3.1.3.js'); ?>
+ _scriptTag('js/legacy.js'); ?>
+ _scriptTag('js/privatebin.js', false); ?>
diff --git a/tpl/bootstrap5.php b/tpl/bootstrap5.php
index 9bf7feb0..3700bd98 100644
--- a/tpl/bootstrap5.php
+++ b/tpl/bootstrap5.php
@@ -25,39 +25,39 @@ if ($SYNTAXHIGHLIGHTING) :
endif;
?>
-
+ _scriptTag('js/jquery-3.7.1.js', false); ?>
-
+ _scriptTag('js/kjua-0.9.0.js'); ?>
-
+ _scriptTag('js/base64-1.7.js'); ?>
-
-
-
-
-
+ _scriptTag('js/zlib-1.3.1.js'); ?>
+ _scriptTag('js/base-x-4.0.0.js'); ?>
+ _scriptTag('js/rawinflate-0.3.js'); ?>
+ _scriptTag('js/bootstrap-5.3.3.js'); ?>
+ _scriptTag('js/dark-mode-switch.js'); ?>
-
+ _scriptTag('js/prettify.js'); ?>
-
+ _scriptTag('js/showdown-2.1.0.js'); ?>
-
-
-
+ _scriptTag('js/purify-3.1.3.js'); ?>
+ _scriptTag('js/legacy.js'); ?>
+ _scriptTag('js/privatebin.js', false); ?>
diff --git a/tpl/page.php b/tpl/page.php
index 47c55203..8a83c9cd 100644
--- a/tpl/page.php
+++ b/tpl/page.php
@@ -21,37 +21,37 @@ if ($SYNTAXHIGHLIGHTING):
endif;
endif;
?>
-
+ _scriptTag('js/jquery-3.7.1.js', false); ?>
-
+ _scriptTag('js/kjua-0.9.0.js'); ?>
-
+ _scriptTag('js/base64-1.7.js'); ?>
-
-
-
+ _scriptTag('js/zlib-1.3.1.js'); ?>
+ _scriptTag('js/base-x-4.0.0.js'); ?>
+ _scriptTag('js/rawinflate-0.3.js'); ?>
-
+ _scriptTag('js/prettify.js'); ?>
-
+ _scriptTag('js/showdown-2.1.0.js'); ?>
-
-
-
+ _scriptTag('js/purify-3.1.3.js'); ?>
+ _scriptTag('js/legacy.js'); ?>
+ _scriptTag('js/privatebin.js', false); ?>
diff --git a/tst/Bootstrap.php b/tst/Bootstrap.php
index 984f7d50..584a8d75 100644
--- a/tst/Bootstrap.php
+++ b/tst/Bootstrap.php
@@ -382,32 +382,21 @@ class Helper
}
}
- $dir = dir(PATH . 'tpl');
- while (false !== ($file = $dir->read())) {
- if (substr($file, -4) === '.php') {
- $content = file_get_contents(
- PATH . 'tpl' . DIRECTORY_SEPARATOR . $file
- );
- $content = preg_replace_callback(
- '##',
- function ($matches) {
- if (array_key_exists($matches[2], Helper::$hashes)) {
- return '';
- } else {
- return $matches[0];
- }
- },
- $content
- );
- file_put_contents(
- PATH . 'tpl' . DIRECTORY_SEPARATOR . $file,
- $content
- );
- }
- }
+ $file = PATH . 'lib' . DIRECTORY_SEPARATOR . 'Configuration.php';
+ $content = preg_replace_callback(
+ '#\'js/([a-z0-9.-]+.js)\' =\> \'([^\']*)\',#',
+ function ($matches) {
+ if (array_key_exists($matches[1], Helper::$hashes)) {
+ return '\'js/' . $matches[1] . '\' => \'sha512-' .
+ Helper::$hashes[$matches[1]] .
+ '\',';
+ } else {
+ return $matches[0];
+ }
+ },
+ file_get_contents($file)
+ );
+ file_put_contents($file, $content);
}
}
diff --git a/tst/ViewTest.php b/tst/ViewTest.php
index 2a1c7e47..65b252de 100644
--- a/tst/ViewTest.php
+++ b/tst/ViewTest.php
@@ -63,6 +63,7 @@ class ViewTest extends TestCase
$page->assign('HTTPSLINK', 'https://example.com/');
$page->assign('COMPRESSION', 'zlib');
$page->assign('CSPHEADER', 'default-src \'none\'');
+ $page->assign('SRI', array());
$dir = dir(PATH . 'tpl');
while (false !== ($file = $dir->read())) {