X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModule%2FLogin.php;h=fadfb206612c3bd728f00efa276b453372ba280a;hb=de8787dd5bb9fadd6388c050bad35ec4c0332e08;hp=f1b41123f4d6ea07f2d697aa3ab0b5fa036ae829;hpb=26b335ef3d8b1ec4e1b4e22cd7d3c34e66d2549d;p=friendica.git
diff --git a/src/Module/Login.php b/src/Module/Login.php
index f1b41123f4..fadfb20661 100644
--- a/src/Module/Login.php
+++ b/src/Module/Login.php
@@ -1,23 +1,28 @@
*/
class Login extends BaseModule
{
@@ -37,7 +42,7 @@ class Login extends BaseModule
goaway(self::getApp()->get_baseurl());
}
- return self::form(self::getApp()->get_baseurl(), $a->config['register_policy'] != REGISTER_CLOSED);
+ return self::form(self::getApp()->get_baseurl(), intval(Config::get('config', 'register_policy')) !== REGISTER_CLOSED);
}
public static function post()
@@ -45,88 +50,117 @@ class Login extends BaseModule
session_unset();
// OpenId Login
if (
- !x($_POST, 'password')
+ empty($_POST['password'])
&& (
- x($_POST, 'openid_url')
- || x($_POST, 'username')
+ !empty($_POST['openid_url'])
+ || !empty($_POST['username'])
)
) {
- $noid = Config::get('system', 'no_openid');
-
- $openid_url = trim($_POST['openid_url'] ? : $_POST['username']);
-
- // if it's an email address or doesn't resolve to a URL, fail.
- if ($noid || strpos($openid_url, '@') || !validate_url($openid_url)) {
- notice(t('Login failed.') . EOL);
- goaway(self::getApp()->get_baseurl());
- // NOTREACHED
- }
+ $openid_url = trim(defaults($_POST, 'openid_url', $_POST['username']));
- // Otherwise it's probably an openid.
- try {
- require_once 'library/openid.php';
- $openid = new LightOpenID;
- $openid->identity = $openid_url;
- $_SESSION['openid'] = $openid_url;
- $_SESSION['remember'] = $_POST['remember'];
- $openid->returnUrl = self::getApp()->get_baseurl(true) . '/openid';
- goaway($openid->authUrl());
- } catch (Exception $e) {
- notice(t('We encountered a problem while logging in with the OpenID you provided. Please check the correct spelling of the ID.') . '
' . t('The error message was:') . ' ' . $e->getMessage());
- }
- // NOTREACHED
+ self::openIdAuthentication($openid_url, !empty($_POST['remember']));
}
if (x($_POST, 'auth-params') && $_POST['auth-params'] === 'login') {
- $record = null;
+ self::passwordAuthentication(
+ trim($_POST['username']),
+ trim($_POST['password']),
+ !empty($_POST['remember'])
+ );
+ }
+ }
- $addon_auth = [
- 'username' => trim($_POST['username']),
- 'password' => trim($_POST['password']),
- 'authenticated' => 0,
- 'user_record' => null
- ];
+ /**
+ * Attempts to authenticate using OpenId
+ *
+ * @param string $openid_url OpenID URL string
+ * @param bool $remember Whether to set the session remember flag
+ */
+ private static function openIdAuthentication($openid_url, $remember)
+ {
+ $noid = Config::get('system', 'no_openid');
- /*
- * A plugin indicates successful login by setting 'authenticated' to non-zero value and returning a user record
- * Plugins should never set 'authenticated' except to indicate success - as hooks may be chained
- * and later plugins should not interfere with an earlier one that succeeded.
- */
- call_hooks('authenticate', $addon_auth);
+ // if it's an email address or doesn't resolve to a URL, fail.
+ if ($noid || strpos($openid_url, '@') || !Network::isUrlValid($openid_url)) {
+ notice(L10n::t('Login failed.') . EOL);
+ goaway(self::getApp()->get_baseurl());
+ // NOTREACHED
+ }
- if ($addon_auth['authenticated'] && count($addon_auth['user_record'])) {
- $record = $addon_auth['user_record'];
- } else {
- $user_id = User::authenticate(trim($_POST['username']), trim($_POST['password']));
- if ($user_id) {
- $record = dba::selectFirst('user', [], ['uid' => $user_id]);
- }
- }
+ // Otherwise it's probably an openid.
+ try {
+ $a = get_app();
+ $openid = new LightOpenID($a->get_hostname());
+ $openid->identity = $openid_url;
+ $_SESSION['openid'] = $openid_url;
+ $_SESSION['remember'] = $remember;
+ $openid->returnUrl = self::getApp()->get_baseurl(true) . '/openid';
+ goaway($openid->authUrl());
+ } catch (Exception $e) {
+ notice(L10n::t('We encountered a problem while logging in with the OpenID you provided. Please check the correct spelling of the ID.') . '
' . L10n::t('The error message was:') . ' ' . $e->getMessage());
+ }
+ }
- if (!$record || !count($record)) {
- logger('authenticate: failed login attempt: ' . notags(trim($_POST['username'])) . ' from IP ' . $_SERVER['REMOTE_ADDR']);
- notice(t('Login failed.') . EOL);
- goaway(self::getApp()->get_baseurl());
- }
+ /**
+ * Attempts to authenticate using login/password
+ *
+ * @param string $username User name
+ * @param string $password Clear password
+ * @param bool $remember Whether to set the session remember flag
+ */
+ private static function passwordAuthentication($username, $password, $remember)
+ {
+ $record = null;
+
+ $addon_auth = [
+ 'username' => $username,
+ 'password' => $password,
+ 'authenticated' => 0,
+ 'user_record' => null
+ ];
+
+ /*
+ * An addon indicates successful login by setting 'authenticated' to non-zero value and returning a user record
+ * Addons should never set 'authenticated' except to indicate success - as hooks may be chained
+ * and later addons should not interfere with an earlier one that succeeded.
+ */
+ Addon::callHooks('authenticate', $addon_auth);
+
+ try {
+ if ($addon_auth['authenticated']) {
+ $record = $addon_auth['user_record'];
- if (!$_POST['remember']) {
- new_cookie(0); // 0 means delete on browser exit
+ if (empty($record)) {
+ throw new Exception(L10n::t('Login failed.'));
+ }
+ } else {
+ $record = DBA::selectFirst('user', [],
+ ['uid' => User::getIdFromPasswordAuthentication($username, $password)]
+ );
}
+ } catch (Exception $e) {
+ logger('authenticate: failed login attempt: ' . notags($username) . ' from IP ' . $_SERVER['REMOTE_ADDR']);
+ notice($e->getMessage() . EOL);
+ goaway(self::getApp()->get_baseurl() . '/login');
+ }
- // if we haven't failed up this point, log them in.
- $_SESSION['remember'] = $_POST['remember'];
- $_SESSION['last_login_date'] = datetime_convert('UTC', 'UTC');
- authenticate_success($record, true, true);
+ if (!$remember) {
+ new_cookie(0); // 0 means delete on browser exit
+ }
- if (x($_SESSION, 'return_url')) {
- $return_url = $_SESSION['return_url'];
- unset($_SESSION['return_url']);
- } else {
- $return_url = '';
- }
+ // if we haven't failed up this point, log them in.
+ $_SESSION['remember'] = $remember;
+ $_SESSION['last_login_date'] = DateTimeFormat::utcNow();
+ authenticate_success($record, true, true);
- goaway($return_url);
+ if (x($_SESSION, 'return_url')) {
+ $return_url = $_SESSION['return_url'];
+ unset($_SESSION['return_url']);
+ } else {
+ $return_url = '';
}
+
+ goaway($return_url);
}
/**
@@ -141,7 +175,7 @@ class Login extends BaseModule
$data = json_decode($_COOKIE["Friendica"]);
if (isset($data->uid)) {
- $user = dba::selectFirst('user', [],
+ $user = DBA::selectFirst('user', [],
[
'uid' => $data->uid,
'blocked' => false,
@@ -150,7 +184,7 @@ class Login extends BaseModule
'verified' => true,
]
);
- if (DBM::is_result($user)) {
+ if (DBA::isResult($user)) {
if ($data->hash != cookie_hash($user)) {
logger("Hash for user " . $data->uid . " doesn't fit.");
nuke_session();
@@ -177,11 +211,9 @@ class Login extends BaseModule
if (isset($_SESSION) && x($_SESSION, 'authenticated')) {
if (x($_SESSION, 'visitor_id') && !x($_SESSION, 'uid')) {
- $r = q("SELECT * FROM `contact` WHERE `id` = %d LIMIT 1",
- intval($_SESSION['visitor_id'])
- );
- if (DBM::is_result($r)) {
- self::getApp()->contact = $r[0];
+ $contact = DBA::selectFirst('contact', [], ['id' => $_SESSION['visitor_id']]);
+ if (DBA::isResult($contact)) {
+ self::getApp()->contact = $contact;
}
}
@@ -196,7 +228,7 @@ class Login extends BaseModule
goaway(self::getApp()->get_baseurl());
}
- $user = dba::selectFirst('user', [],
+ $user = DBA::selectFirst('user', [],
[
'uid' => $_SESSION['uid'],
'blocked' => false,
@@ -205,7 +237,7 @@ class Login extends BaseModule
'verified' => true,
]
);
- if (!DBM::is_result($user)) {
+ if (!DBA::isResult($user)) {
nuke_session();
goaway(self::getApp()->get_baseurl());
}
@@ -213,11 +245,11 @@ class Login extends BaseModule
// Make sure to refresh the last login time for the user if the user
// stays logged in for a long time, e.g. with "Remember Me"
$login_refresh = false;
- if (!x($_SESSION['last_login_date'])) {
- $_SESSION['last_login_date'] = datetime_convert('UTC', 'UTC');
+ if (empty($_SESSION['last_login_date'])) {
+ $_SESSION['last_login_date'] = DateTimeFormat::utcNow();
}
- if (strcmp(datetime_convert('UTC', 'UTC', 'now - 12 hours'), $_SESSION['last_login_date']) > 0) {
- $_SESSION['last_login_date'] = datetime_convert('UTC', 'UTC');
+ if (strcmp(DateTimeFormat::utc('now - 12 hours'), $_SESSION['last_login_date']) > 0) {
+ $_SESSION['last_login_date'] = DateTimeFormat::utcNow();
$login_refresh = true;
}
authenticate_success($user, false, false, $login_refresh);
@@ -231,7 +263,7 @@ class Login extends BaseModule
* @param string $return_url The url relative to the base the user should be sent
* back to after login completes
* @param bool $register If $register == true provide a registration link.
- * This will most always depend on the value of $a->config['register_policy'].
+ * This will most always depend on the value of config.register_policy.
* @param array $hiddens optional
*
* @return string Returns the complete html for inserting into the page
@@ -245,8 +277,8 @@ class Login extends BaseModule
$reg = false;
if ($register) {
$reg = [
- 'title' => t('Create a New Account'),
- 'desc' => t('Register')
+ 'title' => L10n::t('Create a New Account'),
+ 'desc' => L10n::t('Register')
];
}
@@ -274,32 +306,32 @@ class Login extends BaseModule
$tpl,
[
'$dest_url' => self::getApp()->get_baseurl(true) . '/login',
- '$logout' => t('Logout'),
- '$login' => t('Login'),
+ '$logout' => L10n::t('Logout'),
+ '$login' => L10n::t('Login'),
- '$lname' => ['username', t('Nickname or Email: ') , '', ''],
- '$lpassword' => ['password', t('Password: '), '', ''],
- '$lremember' => ['remember', t('Remember me'), 0, ''],
+ '$lname' => ['username', L10n::t('Nickname or Email: ') , '', ''],
+ '$lpassword' => ['password', L10n::t('Password: '), '', ''],
+ '$lremember' => ['remember', L10n::t('Remember me'), 0, ''],
'$openid' => !$noid,
- '$lopenid' => ['openid_url', t('Or login using OpenID: '),'',''],
+ '$lopenid' => ['openid_url', L10n::t('Or login using OpenID: '),'',''],
'$hiddens' => $hiddens,
'$register' => $reg,
- '$lostpass' => t('Forgot your password?'),
- '$lostlink' => t('Password Reset'),
+ '$lostpass' => L10n::t('Forgot your password?'),
+ '$lostlink' => L10n::t('Password Reset'),
- '$tostitle' => t('Website Terms of Service'),
- '$toslink' => t('terms of service'),
+ '$tostitle' => L10n::t('Website Terms of Service'),
+ '$toslink' => L10n::t('terms of service'),
- '$privacytitle' => t('Website Privacy Policy'),
- '$privacylink' => t('privacy policy'),
+ '$privacytitle' => L10n::t('Website Privacy Policy'),
+ '$privacylink' => L10n::t('privacy policy'),
]
);
- call_hooks('login_hook', $o);
+ Addon::callHooks('login_hook', $o);
return $o;
}