X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fopenid.php;h=d1404ba804f4afec2f65de81db24d96a7ec9ff4e;hb=f2ca3e5be44192c486e8e3af2a993e065ad40a7d;hp=b45cd97975e3f7b838b3eb2283cafcd74a4d9f3c;hpb=6fdc219143b83c357a65179529752553a71b44f6;p=friendica.git diff --git a/mod/openid.php b/mod/openid.php index b45cd97975..d1404ba804 100644 --- a/mod/openid.php +++ b/mod/openid.php @@ -1,28 +1,35 @@ internalRedirect(); - logger('mod_openid ' . print_r($_REQUEST,true), LOGGER_DATA); + Logger::log('mod_openid ' . print_r($_REQUEST,true), Logger::DATA); if((x($_GET,'openid_mode')) && (x($_SESSION,'openid'))) { - $openid = new LightOpenID; + $openid = new LightOpenID($a->getHostName()); if($openid->validate()) { $authid = $_REQUEST['openid_identity']; if(! strlen($authid)) { - logger( t('OpenID protocol error. No ID returned.') . EOL); - goaway(z_root()); + Logger::log(L10n::t('OpenID protocol error. No ID returned.') . EOL); + $a->internalRedirect(); } // NOTE: we search both for normalised and non-normalised form of $authid @@ -30,35 +37,35 @@ function openid_content(App $a) { // mod/settings.php in 8367cad so it might have left mixed // records in the user table // - $r = q("SELECT *, `user`.`pubkey` as `upubkey`, `user`.`prvkey` as `uprvkey` 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 LIMIT 1", - dbesc($authid), dbesc(normalise_openid($authid)) + DBA::escape($authid), DBA::escape(normalise_openid($authid)) ); - if (dbm::is_result($r)) { + if (DBA::isResult($r)) { // successful OpenID login unset($_SESSION['openid']); - require_once('include/security.php'); - authenticate_success($r[0],true,true); + Authentication::setAuthenticatedSessionForUser($r[0],true,true); // just in case there was no return url set // and we fell through - goaway(z_root()); + $a->internalRedirect(); } // 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 (intval(Config::get('config', 'register_policy')) === REGISTER_CLOSED) { + notice(L10n::t('Account not found and OpenID registration is not permitted on this site.') . EOL); + $a->internalRedirect(); } unset($_SESSION['register']); @@ -73,10 +80,10 @@ function openid_content(App $a) { $first = notags(trim($v)); } if($k === 'namePerson') { - $args .= '&username=' . notags(trim($v)); + $args .= '&username=' . urlencode(notags(trim($v))); } if ($k === 'contact/email') { - $args .= '&email=' . notags(trim($v)); + $args .= '&email=' . urlencode(notags(trim($v))); } if ($k === 'media/image/aspect11') { $photosq = bin2hex(trim($v)); @@ -87,27 +94,27 @@ function openid_content(App $a) { } } if ($nick) { - $args .= '&nickname=' . $nick; + $args .= '&nickname=' . urlencode($nick); } elseif ($first) { - $args .= '&nickname=' . $first; + $args .= '&nickname=' . urlencode($first); } if ($photosq) { - $args .= '&photo=' . $photosq; + $args .= '&photo=' . urlencode($photosq); } elseif ($photo) { - $args .= '&photo=' . $photo; + $args .= '&photo=' . urlencode($photo); } - $args .= '&openid_url=' . notags(trim($authid)); + $args .= '&openid_url=' . urlencode(notags(trim($authid))); - goaway(App::get_baseurl() . '/register' . $args); + $a->internalRedirect('register?' . $args); // NOTREACHED } } - notice( t('Login failed.') . EOL); - goaway(z_root()); + notice(L10n::t('Login failed.') . EOL); + $a->internalRedirect(); // NOTREACHED }