OpenId possibly fixed:

- The LightOpenId class' constructor now requires the hostname from where the
  requested started, added it now
- sometimes need to invoke get_app() as $a was not around

Signed-off-by: Roland Häder <roland@mxchange.org>

diff --git a/mod/openid.php b/mod/openid.php
index af7016c32b0dcd640566685786345f1e840026d3..2c284f76d88757848459a339cfbe98bdafcfdcb9 100644 (file)
--- a/mod/openid.php
+++ b/mod/openid.php
@@ -19,7 +19,7 @@ function openid_content(App $a) {
 
        if((x($_GET,'openid_mode')) && (x($_SESSION,'openid'))) {
 
-               $openid = new LightOpenID;
+               $openid = new LightOpenID($a->get_hostname());
 
                if($openid->validate()) {
 

diff --git a/mod/settings.php b/mod/settings.php
index c7e61cbe3b3fcd30072fda802e88898b848a1906..23869913acedb2b2d8882461f9467020678d819e 100644 (file)
--- a/mod/settings.php
+++ b/mod/settings.php
@@ -543,7 +543,7 @@ function settings_post(App $a)
        if ($openid != $a->user['openid'] || (strlen($openid) && (!strlen($openidserver)))) {
                if (Network::isUrlValid($openid)) {
                        logger('updating openidserver');
-                       $open_id_obj = new LightOpenID;
+                       $open_id_obj = new LightOpenID($a->get_hostname());
                        $open_id_obj->identity = $openid;
                        $openidserver = $open_id_obj->discover($open_id_obj->identity);
                } else {

diff --git a/src/Model/User.php b/src/Model/User.php
index 41d26ee19d66f4c27d3b3a885f086564d9e466c0..9b0f409e11b150b6b39e279655a5553c29fdcb7f 100644 (file)
--- a/src/Model/User.php
+++ b/src/Model/User.php
@@ -359,7 +359,7 @@ class User
                                $_SESSION['register'] = 1;
                                $_SESSION['openid'] = $openid_url;
 
-                               $openid = new LightOpenID;
+                               $openid = new LightOpenID($a->get_hostname());
                                $openid->identity = $openid_url;
                                $openid->returnUrl = System::baseUrl() . '/openid';
                                $openid->required = ['namePerson/friendly', 'contact/email', 'namePerson'];

diff --git a/src/Module/Login.php b/src/Module/Login.php
index fe3305a91b8846a893ef279583b41cc14c01563e..7d6470620acfda4f5b419b8b5b17452a23f8894b 100644 (file)
--- a/src/Module/Login.php
+++ b/src/Module/Login.php
@@ -90,7 +90,8 @@ class Login extends BaseModule
 
                // Otherwise it's probably an openid.
                try {
-                       $openid = new LightOpenID;
+                       $a = get_app();
+                       $openid = new LightOpenID($a->get_hostname());
                        $openid->identity = $openid_url;
                        $_SESSION['openid'] = $openid_url;
                        $_SESSION['remember'] = $remember;