X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fopenid.php;h=41d45c1f608a887889abc087baa6119f40b67113;hb=7529fc61dd9169368224a092bcdf7d3cfeee1982;hp=a92a124c0d06a8978dd72f5b9cbaede0cf43d20b;hpb=0c67c89715df801e0c887be50c9739db3bae7ca3;p=friendica.git

diff --git a/mod/openid.php b/mod/openid.php
index a92a124c0d..41d45c1f60 100644
--- a/mod/openid.php
+++ b/mod/openid.php
@@ -1,36 +1,50 @@
 <?php
+/**
+ * @file mod/openid.php
+ */
 
-require_once('library/openid.php');
+use Friendica\App;
+use Friendica\Core\Config;
+use Friendica\Core\L10n;
+use Friendica\Core\System;
+use Friendica\Database\DBA;
 
-if(! function_exists('openid_content')) {
-function openid_content(&$a) {
+function openid_content(App $a) {
 
-	$noid = get_config('system','no_openid');
+	$noid = Config::get('system','no_openid');
 	if($noid)
-		goaway(z_root());
+		goaway(System::baseUrl());
 
 	logger('mod_openid ' . print_r($_REQUEST,true), LOGGER_DATA);
 
 	if((x($_GET,'openid_mode')) && (x($_SESSION,'openid'))) {
 
-		$openid = new LightOpenID;
+		$openid = new LightOpenID($a->get_hostname());
 
 		if($openid->validate()) {
 
-			$authid = normalise_openid($_REQUEST['openid_identity']);
+			$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());
 			}
 
-			$r = q("SELECT `user`.*, `user`.`pubkey` as `upubkey`, `user`.`prvkey` as `uprvkey`
-				FROM `user` WHERE `openid` = '%s' AND `blocked` = 0
-				AND `account_expired` = 0 AND `account_removed` = 0 AND `verified` = 1 LIMIT 1",
-				dbesc($authid)
+			// NOTE: we search both for normalised and non-normalised form of $authid
+			//       because the normalization step was removed from setting
+			//       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(normalise_openid($authid))
 			);
 
-			if($r && count($r)) {
+			if (DBA::isResult($r)) {
 
 				// successful OpenID login
 
@@ -42,55 +56,64 @@ function openid_content(&$a) {
 				// 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 (intval(Config::get('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($a->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
 }
-}