X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fopenid.php;h=2c284f76d88757848459a339cfbe98bdafcfdcb9;hb=132653ac562d28bdc10eb836628daec8175cd378;hp=e14b5f82d93f57e4e56c6516712cff5490e6b1cc;hpb=4dce3d822470537e40a0f77330d62ac05bf73806;p=friendica.git diff --git a/mod/openid.php b/mod/openid.php index e14b5f82d9..2c284f76d8 100644 --- a/mod/openid.php +++ b/mod/openid.php @@ -1,28 +1,33 @@ get_hostname()); if($openid->validate()) { $authid = $_REQUEST['openid_identity']; if(! strlen($authid)) { - logger( t('OpenID protocol error. No ID returned.') . EOL); - goaway(z_root()); + logger(L10n::t('OpenID protocol error. No ID returned.') . EOL); + goaway(System::baseUrl()); } // NOTE: we search both for normalised and non-normalised form of $authid @@ -30,7 +35,8 @@ function openid_content(App &$a) { // mod/settings.php in 8367cad so it might have left mixed // records in the user table // - $r = q("SELECT * FROM `user` + $r = q("SELECT * + FROM `user` WHERE ( `openid` = '%s' OR `openid` = '%s' ) AND `blocked` = 0 AND `account_expired` = 0 AND `account_removed` = 0 AND `verified` = 1 @@ -38,7 +44,7 @@ function openid_content(App &$a) { dbesc($authid), dbesc(normalise_openid($authid)) ); - if (dbm::is_result($r)) { + if (DBM::is_result($r)) { // successful OpenID login @@ -47,57 +53,67 @@ function openid_content(App &$a) { require_once('include/security.php'); authenticate_success($r[0],true,true); - // just in case there was no return url set + // just in case there was no return url set // and we fell through - goaway(z_root()); + goaway(System::baseUrl()); } // Successful OpenID login - but we can't match it to an existing account. // New registration? - if($a->config['register_policy'] == REGISTER_CLOSED) { - notice( t('Account not found and OpenID registration is not permitted on this site.') . EOL); - goaway(z_root()); + if ($a->config['register_policy'] == REGISTER_CLOSED) { + notice(L10n::t('Account not found and OpenID registration is not permitted on this site.') . EOL); + goaway(System::baseUrl()); } unset($_SESSION['register']); $args = ''; $attr = $openid->getAttributes(); - if(is_array($attr) && count($attr)) { - foreach($attr as $k => $v) { - if($k === 'namePerson/friendly') + if (is_array($attr) && count($attr)) { + foreach ($attr as $k => $v) { + if ($k === 'namePerson/friendly') { $nick = notags(trim($v)); - if($k === 'namePerson/first') + } + if($k === 'namePerson/first') { $first = notags(trim($v)); - if($k === 'namePerson') - $args .= '&username=' . notags(trim($v)); - if($k === 'contact/email') - $args .= '&email=' . notags(trim($v)); - if($k === 'media/image/aspect11') + } + if($k === 'namePerson') { + $args .= '&username=' . urlencode(notags(trim($v))); + } + if ($k === 'contact/email') { + $args .= '&email=' . urlencode(notags(trim($v))); + } + if ($k === 'media/image/aspect11') { $photosq = bin2hex(trim($v)); - if($k === 'media/image/default') + } + if ($k === 'media/image/default') { $photo = bin2hex(trim($v)); + } } } - if($nick) - $args .= '&nickname=' . $nick; - elseif($first) - $args .= '&nickname=' . $first; + if ($nick) { + $args .= '&nickname=' . urlencode($nick); + } + elseif ($first) { + $args .= '&nickname=' . urlencode($first); + } - if($photosq) - $args .= '&photo=' . $photosq; - elseif($photo) - $args .= '&photo=' . $photo; + if ($photosq) { + $args .= '&photo=' . urlencode($photosq); + } + elseif ($photo) { + $args .= '&photo=' . urlencode($photo); + } - $args .= '&openid_url=' . notags(trim($authid)); + $args .= '&openid_url=' . urlencode(notags(trim($authid))); - goaway(App::get_baseurl() . '/register' . $args); + goaway(System::baseUrl() . '/register?' . $args); // NOTREACHED } } - notice( t('Login failed.') . EOL); - goaway(z_root()); + notice(L10n::t('Login failed.') . EOL); + goaway(System::baseUrl()); // NOTREACHED }