Issue 3229: Switched to other function to receive the identity

This commit is contained in:
Michael 2019-09-11 16:46:13 +00:00
parent 7a9aba7674
commit f71abb30ca

View File

@ -13,9 +13,9 @@ use Friendica\Util\Strings;
function openid_content(App $a) {
$noid = Config::get('system','no_openid');
if($noid)
if (Config::get('system','no_openid')) {
$a->internalRedirect();
}
Logger::log('mod_openid ' . print_r($_REQUEST,true), Logger::DATA);
@ -24,10 +24,9 @@ function openid_content(App $a) {
$openid = new LightOpenID($a->getHostName());
if ($openid->validate()) {
$authid = $openid->identity;
$authid = $_REQUEST['openid_identity'];
if(! strlen($authid)) {
if (empty($authid)) {
Logger::log(L10n::t('OpenID protocol error. No ID returned.') . EOL);
$a->internalRedirect();
}
@ -37,22 +36,16 @@ 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`
WHERE ( `openid` = '%s' OR `openid` = '%s' )
AND `blocked` = 0 AND `account_expired` = 0
AND `account_removed` = 0 AND `verified` = 1
LIMIT 1",
DBA::escape($authid), DBA::escape(Strings::normaliseOpenID($authid))
);
if (DBA::isResult($r)) {
$condition = ['blocked' => false, 'account_expired' => false, 'account_removed' => false, 'verified' => true,
'openid' => [$authid, Strings::normaliseOpenID($authid)]];
$user = DBA::selectFirst('user', [], $condition);
if (DBA::isResult($user)) {
// successful OpenID login
unset($_SESSION['openid']);
Session::setAuthenticatedForUser($a, $r[0],true,true);
Session::setAuthenticatedForUser($a, $user, true, true);
// just in case there was no return url set
// and we fell through
@ -95,15 +88,13 @@ function openid_content(App $a) {
}
if (!empty($nick)) {
$args .= '&nickname=' . urlencode($nick);
}
elseif (!empty($first)) {
} elseif (!empty($first)) {
$args .= '&nickname=' . urlencode($first);
}
if (!empty($photosq)) {
$args .= '&photo=' . urlencode($photosq);
}
elseif (!empty($photo)) {
} elseif (!empty($photo)) {
$args .= '&photo=' . urlencode($photo);
}