Skip to content

Commit b8805ef

Browse files
committed
chore: add debug info
fix: check non empty Change-Id: I243d0b6162ff8271d63c639094891a079577c61c
1 parent 8fa81ba commit b8805ef

File tree

1 file changed

+26
-5
lines changed

1 file changed

+26
-5
lines changed

app/libs/OAuth2/Strategies/LoginHintProcessStrategy.php

Lines changed: 26 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,25 +50,46 @@ public function __construct(IAuthService $auth_service, ISecurityContextService
5050
* @return int|string|null
5151
*/
5252
private function getLoginHint(){
53+
5354
$ctx = $this->security_context_service->get();
54-
$Login_hint = null;
55+
$login_hint = null;
5556
if(!is_null($ctx)) {
56-
$Login_hint = $ctx->getRequestedUserId();
57+
$login_hint = $ctx->getRequestedUserId();
5758
}
5859

59-
if(is_null($Login_hint)){
60+
if(empty($login_hint)){
61+
Log::debug
62+
(
63+
sprintf
64+
(
65+
"LoginHintProcessStrategy::getLoginHint no login hint in security context, trying to get from QS"
66+
)
67+
);
68+
6069
if(Request::has(OAuth2Protocol::OAuth2Protocol_LoginHint)) {
6170
$login_hint = Request::query(OAuth2Protocol::OAuth2Protocol_LoginHint);
62-
if (!EmailUtils::isValidEmail($login_hint))
71+
Log::debug(sprintf("LoginHintProcessStrategy::getLoginHint login_hint %s from QS", $login_hint));
72+
if (!EmailUtils::isValidEmail($login_hint)) {
73+
Log::debug
74+
(
75+
sprintf
76+
(
77+
"LoginHintProcessStrategy::getLoginHint login_hint %s is not a valid email",
78+
$login_hint
79+
)
80+
);
81+
6382
$login_hint = null;
83+
}
6484
}
6585
}
6686

67-
return $Login_hint;
87+
return $login_hint;
6888
}
6989

7090
public function process():void{
7191
$login_hint = $this->getLoginHint();
92+
Log::debug(sprintf("LoginHintProcessStrategy::process login_hint %s", $login_hint));
7293
// login hint processing
7394
Session::forget(['username', 'user_fullname', 'user_pic', 'user_verified']);
7495
if (!is_null($login_hint)) {

0 commit comments

Comments
 (0)