Skip to content

Commit

Permalink
tests: use non-empty string for static salt
Browse files Browse the repository at this point in the history
As pointed out by @Klap-in, an empty string may evaluate to false in
some circumstances. This is something we may not want. Using a string
like 'test' should therefore be more robust.
  • Loading branch information
micgro42 committed Mar 19, 2018
1 parent a1fe3c9 commit fe745be
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion inc/auth.php
Expand Up @@ -309,7 +309,7 @@ function auth_browseruid() {
*/
function auth_cookiesalt($addsession = false, $secure = false) {
if (defined('SIMPLE_TEST')) {
return '';
return 'test';
}
global $conf;
$file = $conf['metadir'].'/_htcookiesalt';
Expand Down

0 comments on commit fe745be

Please sign in to comment.