chore: add debug info

fix: check non empty

Change-Id: I243d0b6162ff8271d63c639094891a079577c61c

Author: smarcet

app/libs/OAuth2/Strategies/LoginHintProcessStrategy.php

@@ -50,17 +50,36 @@ public function __construct(IAuthService $auth_service, ISecurityContextService
      * @return int|string|null
      */
     private function getLoginHint(){
+
         $ctx = $this->security_context_service->get();
         $Login_hint = null;
         if(!is_null($ctx)) {
             $Login_hint = $ctx->getRequestedUserId();
         }
 
-        if(is_null($Login_hint)){
+        if(empty($Login_hint)){
+            Log::debug
+            (
+                sprintf
+                (
+                    "LoginHintProcessStrategy::getLoginHint no login hint in security context, trying to get from QS"
+                )
+            );
+
             if(Request::has(OAuth2Protocol::OAuth2Protocol_LoginHint)) {
                 $login_hint = Request::query(OAuth2Protocol::OAuth2Protocol_LoginHint);
-                if (!EmailUtils::isValidEmail($login_hint))
+                Log::debug(sprintf("LoginHintProcessStrategy::getLoginHint login_hint %s from QS", $login_hint));
+                if (!EmailUtils::isValidEmail($login_hint)) {
+                    Log::debug
+                    (
+                        sprintf
+                        (
+                            "LoginHintProcessStrategy::getLoginHint login_hint %s is not a valid email",
+                            $login_hint
+                        )
+                    );
                     $login_hint = null;
+                }
             }
         }
 
@@ -69,6 +88,7 @@ private function getLoginHint(){
 
     public function process():void{
         $login_hint = $this->getLoginHint();
+        Log::debug(sprintf("LoginHintProcessStrategy::process login_hint %s", $login_hint));
         // login hint processing
         Session::forget(['username', 'user_fullname', 'user_pic', 'user_verified']);
         if (!is_null($login_hint)) {