Merge pull request #11812 from annando/ap-performance

[friendica.git] / src / Module / Security / OpenID.php

diff --git a/src/Module/Security/OpenID.php b/src/Module/Security/OpenID.php
index bd0df0a4e71699fef2e93b82a73080144094af10..7dbb765c6d83bb3b5763eeaae3acc523f9bd3a1b 100644 (file)
--- a/src/Module/Security/OpenID.php
+++ b/src/Module/Security/OpenID.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -31,7 +31,7 @@ use LightOpenID;
  */
 class OpenID extends BaseModule
 {
-       public static function content(array $parameters = [])
+       protected function content(array $request = []): string
        {
                if (DI::config()->get('system', 'no_openid')) {
                        DI::baseUrl()->redirect();
@@ -73,9 +73,7 @@ class OpenID extends BaseModule
 
                                        DI::auth()->setForUser(DI::app(), $user, true, true);
 
-                                       // just in case there was no return url set
-                                       // and we fell through
-                                       DI::baseUrl()->redirect();
+                                       $this->baseUrl->redirect(DI::session()->pop('return_path', ''));
                                }
 
                                // Successful OpenID login - but we can't match it to an existing account.
@@ -84,7 +82,7 @@ class OpenID extends BaseModule
                                $session->set('openid_identity', $authId);
 
                                // Detect the server URL
-                               $open_id_obj = new LightOpenID(DI::baseUrl()->getHostName());
+                               $open_id_obj = new LightOpenID(DI::baseUrl()->getHostname());
                                $open_id_obj->identity = $authId;
                                $session->set('openid_server', $open_id_obj->discover($open_id_obj->identity));
 
@@ -97,5 +95,7 @@ class OpenID extends BaseModule
                                DI::baseUrl()->redirect('login');
                        }
                }
+
+               return '';
        }
 }