Skip to content
This repository was archived by the owner on Sep 19, 2022. It is now read-only.

Commit 66fa586

Browse files
committed
Fix bad log messages for USERONLY mode.
1 parent 507e6b3 commit 66fa586

File tree

2 files changed

+21
-7
lines changed

2 files changed

+21
-7
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22
All notable changes to this project will be documented in this file.
33

44
## [Unreleased]
5+
#### Fixed
6+
- Fixed bad log message in PerunIdentity in mode USERONLY
57

68
## [v4.1.0]
79
#### Changed

lib/Auth/Process/PerunIdentity.php

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -243,14 +243,26 @@ public function process(&$request)
243243
$this->unauthorized($request);
244244
}
245245
}
246-
}
247-
248246

249-
Logger::info(
250-
'Perun user with identity/ies: ' . implode(',', $uids) .
251-
' has been found and SP has sufficient rights to get info about him. ' .
252-
'User ' . $user->getName() . ' with id: ' . $user->getId() . ' is being set to request'
253-
);
247+
Logger::info(
248+
'Perun user with identity/ies: ' . implode(',', $uids) .
249+
' has been found and SP has sufficient rights to get info about him. ' .
250+
'User ' . $user->getName() . ' with id: ' . $user->getId() . ' is being set to request'
251+
);
252+
} elseif ($this->mode === self::MODE_USERONLY) {
253+
if (isset($user)) {
254+
Logger::info(
255+
'Perun user with identity/ies: ' . implode(',', $uids) .
256+
' has been found in mode USERONLY ' .
257+
'User ' . $user->getName() . ' with id: ' . $user->getId() . ' is being set to request'
258+
);
259+
} else {
260+
Logger::info(
261+
'Perun user with identity/ies: ' . implode(',', $uids) .
262+
' has not been found in mode USERONLY. Redirecting to SP.'
263+
);
264+
}
265+
}
254266

255267
if (!isset($request['perun'])) {
256268
$request['perun'] = [];

0 commit comments

Comments
 (0)