Skip to content

Commit 1258c6c

Browse files
committed
do not require password in session. replaces #135
We do not really care for the password in the session and usually don't even have it when authenticating via oAuth. This fixes a warning on PHP 8+
1 parent da47332 commit 1258c6c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Session.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public function setUser($userdata, $resettime = true)
9696
$_SERVER['REMOTE_USER'] = $userdata['user'];
9797

9898
$_SESSION[DOKU_COOKIE]['auth']['user'] = $userdata['user'];
99-
$_SESSION[DOKU_COOKIE]['auth']['pass'] = $userdata['pass'];
99+
$_SESSION[DOKU_COOKIE]['auth']['pass'] = 'not-set'; // pass is neither needed nor wanted
100100
$_SESSION[DOKU_COOKIE]['auth']['info'] = $USERINFO;
101101
$_SESSION[DOKU_COOKIE]['auth']['buid'] = auth_browseruid();
102102
if ($resettime) {

0 commit comments

Comments
 (0)