2017-10-25 23:03:31 +02:00
|
|
|
<?php
|
2018-04-27 22:12:22 +02:00
|
|
|
namespace Shaarli\Security;
|
2017-10-25 23:03:31 +02:00
|
|
|
|
|
|
|
require_once 'tests/utils/FakeConfigManager.php';
|
|
|
|
use \PHPUnit\Framework\TestCase;
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Test coverage for LoginManager
|
|
|
|
*/
|
|
|
|
class LoginManagerTest extends TestCase
|
|
|
|
{
|
2018-05-06 17:12:48 +02:00
|
|
|
/** @var \FakeConfigManager Configuration Manager instance */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $configManager = null;
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/** @var LoginManager Login Manager instance */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $loginManager = null;
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/** @var SessionManager Session Manager instance */
|
|
|
|
protected $sessionManager = null;
|
|
|
|
|
|
|
|
/** @var string Banned IP filename */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $banFile = 'sandbox/ipbans.php';
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/** @var string Log filename */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $logFile = 'sandbox/shaarli.log';
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/** @var array Simulates the $_COOKIE array */
|
|
|
|
protected $cookie = [];
|
|
|
|
|
|
|
|
/** @var array Simulates the $GLOBALS array */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $globals = [];
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/** @var array Simulates the $_SERVER array */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $server = [];
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/** @var array Simulates the $_SESSION array */
|
|
|
|
protected $session = [];
|
|
|
|
|
|
|
|
/** @var string Advertised client IP address */
|
|
|
|
protected $clientIpAddress = '10.1.47.179';
|
|
|
|
|
|
|
|
/** @var string Local client IP address */
|
|
|
|
protected $ipAddr = '127.0.0.1';
|
|
|
|
|
|
|
|
/** @var string Trusted proxy IP address */
|
2017-10-25 23:03:31 +02:00
|
|
|
protected $trustedProxy = '10.1.1.100';
|
|
|
|
|
2018-05-06 17:06:36 +02:00
|
|
|
/** @var string User login */
|
|
|
|
protected $login = 'johndoe';
|
|
|
|
|
|
|
|
/** @var string User password */
|
|
|
|
protected $password = 'IC4nHazL0g1n?';
|
|
|
|
|
|
|
|
/** @var string Hash of the salted user password */
|
|
|
|
protected $passwordHash = '';
|
|
|
|
|
|
|
|
/** @var string Salt used by hash functions */
|
|
|
|
protected $salt = '669e24fa9c5a59a613f98e8e38327384504a4af2';
|
|
|
|
|
2017-10-25 23:03:31 +02:00
|
|
|
/**
|
|
|
|
* Prepare or reset test resources
|
|
|
|
*/
|
|
|
|
public function setUp()
|
|
|
|
{
|
|
|
|
if (file_exists($this->banFile)) {
|
|
|
|
unlink($this->banFile);
|
|
|
|
}
|
|
|
|
|
2018-05-06 17:06:36 +02:00
|
|
|
$this->passwordHash = sha1($this->password . $this->login . $this->salt);
|
|
|
|
|
2017-10-25 23:03:31 +02:00
|
|
|
$this->configManager = new \FakeConfigManager([
|
2018-05-06 17:06:36 +02:00
|
|
|
'credentials.login' => $this->login,
|
|
|
|
'credentials.hash' => $this->passwordHash,
|
|
|
|
'credentials.salt' => $this->salt,
|
2017-10-25 23:03:31 +02:00
|
|
|
'resource.ban_file' => $this->banFile,
|
|
|
|
'resource.log' => $this->logFile,
|
|
|
|
'security.ban_after' => 4,
|
|
|
|
'security.ban_duration' => 3600,
|
|
|
|
'security.trusted_proxies' => [$this->trustedProxy],
|
|
|
|
]);
|
|
|
|
|
2018-05-06 17:12:48 +02:00
|
|
|
$this->cookie = [];
|
|
|
|
|
2017-10-25 23:03:31 +02:00
|
|
|
$this->globals = &$GLOBALS;
|
|
|
|
unset($this->globals['IPBANS']);
|
|
|
|
|
2018-05-30 02:09:09 +02:00
|
|
|
$this->session = [];
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
$this->sessionManager = new SessionManager($this->session, $this->configManager);
|
|
|
|
$this->loginManager = new LoginManager($this->globals, $this->configManager, $this->sessionManager);
|
2017-10-25 23:03:31 +02:00
|
|
|
$this->server['REMOTE_ADDR'] = $this->ipAddr;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Wipe test resources
|
|
|
|
*/
|
|
|
|
public function tearDown()
|
|
|
|
{
|
|
|
|
unset($this->globals['IPBANS']);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Instantiate a LoginManager and load ban records
|
|
|
|
*/
|
|
|
|
public function testReadBanFile()
|
|
|
|
{
|
|
|
|
file_put_contents(
|
|
|
|
$this->banFile,
|
|
|
|
"<?php\n\$GLOBALS['IPBANS']=array('FAILURES' => array('127.0.0.1' => 99));\n?>"
|
|
|
|
);
|
2018-02-17 01:46:27 +01:00
|
|
|
new LoginManager($this->globals, $this->configManager, null);
|
2017-10-25 23:03:31 +02:00
|
|
|
$this->assertEquals(99, $this->globals['IPBANS']['FAILURES']['127.0.0.1']);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Record a failed login attempt
|
|
|
|
*/
|
|
|
|
public function testHandleFailedLogin()
|
|
|
|
{
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(1, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(2, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Record a failed login attempt - IP behind a trusted proxy
|
|
|
|
*/
|
|
|
|
public function testHandleFailedLoginBehindTrustedProxy()
|
|
|
|
{
|
|
|
|
$server = [
|
|
|
|
'REMOTE_ADDR' => $this->trustedProxy,
|
|
|
|
'HTTP_X_FORWARDED_FOR' => $this->ipAddr,
|
|
|
|
];
|
|
|
|
$this->loginManager->handleFailedLogin($server);
|
|
|
|
$this->assertEquals(1, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
|
|
|
|
$this->loginManager->handleFailedLogin($server);
|
|
|
|
$this->assertEquals(2, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Record a failed login attempt - IP behind a trusted proxy but not forwarded
|
|
|
|
*/
|
|
|
|
public function testHandleFailedLoginBehindTrustedProxyNoIp()
|
|
|
|
{
|
|
|
|
$server = [
|
|
|
|
'REMOTE_ADDR' => $this->trustedProxy,
|
|
|
|
];
|
|
|
|
$this->loginManager->handleFailedLogin($server);
|
|
|
|
$this->assertFalse(isset($this->globals['IPBANS']['FAILURES'][$this->ipAddr]));
|
|
|
|
|
|
|
|
$this->loginManager->handleFailedLogin($server);
|
|
|
|
$this->assertFalse(isset($this->globals['IPBANS']['FAILURES'][$this->ipAddr]));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Record a failed login attempt and ban the IP after too many failures
|
|
|
|
*/
|
|
|
|
public function testHandleFailedLoginBanIp()
|
|
|
|
{
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(1, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(2, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(3, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(4, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
$this->assertFalse($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
// handleFailedLogin is not supposed to be called at this point:
|
|
|
|
// - no login form should be displayed once an IP has been banned
|
|
|
|
// - yet this could happen when using custom templates / scripts
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(5, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
$this->assertFalse($this->loginManager->canLogin($this->server));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Nothing to do
|
|
|
|
*/
|
|
|
|
public function testHandleSuccessfulLogin()
|
|
|
|
{
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
$this->loginManager->handleSuccessfulLogin($this->server);
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Erase failure records after successfully logging in from this IP
|
|
|
|
*/
|
|
|
|
public function testHandleSuccessfulLoginAfterFailure()
|
|
|
|
{
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->loginManager->handleFailedLogin($this->server);
|
|
|
|
$this->assertEquals(2, $this->globals['IPBANS']['FAILURES'][$this->ipAddr]);
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
$this->loginManager->handleSuccessfulLogin($this->server);
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
$this->assertFalse(isset($this->globals['IPBANS']['FAILURES'][$this->ipAddr]));
|
|
|
|
$this->assertFalse(isset($this->globals['IPBANS']['BANS'][$this->ipAddr]));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* The IP is not banned
|
|
|
|
*/
|
|
|
|
public function testCanLoginIpNotBanned()
|
|
|
|
{
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* The IP is banned
|
|
|
|
*/
|
|
|
|
public function testCanLoginIpBanned()
|
|
|
|
{
|
|
|
|
// ban the IP for an hour
|
|
|
|
$this->globals['IPBANS']['FAILURES'][$this->ipAddr] = 10;
|
|
|
|
$this->globals['IPBANS']['BANS'][$this->ipAddr] = time() + 3600;
|
|
|
|
|
|
|
|
$this->assertFalse($this->loginManager->canLogin($this->server));
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* The IP is banned, and the ban duration is over
|
|
|
|
*/
|
|
|
|
public function testCanLoginIpBanExpired()
|
|
|
|
{
|
|
|
|
// ban the IP for an hour
|
|
|
|
$this->globals['IPBANS']['FAILURES'][$this->ipAddr] = 10;
|
|
|
|
$this->globals['IPBANS']['BANS'][$this->ipAddr] = time() + 3600;
|
|
|
|
$this->assertFalse($this->loginManager->canLogin($this->server));
|
|
|
|
|
|
|
|
// lift the ban
|
|
|
|
$this->globals['IPBANS']['BANS'][$this->ipAddr] = time() - 3600;
|
|
|
|
$this->assertTrue($this->loginManager->canLogin($this->server));
|
|
|
|
}
|
2018-05-06 17:06:36 +02:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Generate a token depending on the user credentials and client IP
|
|
|
|
*/
|
|
|
|
public function testGenerateStaySignedInToken()
|
|
|
|
{
|
2018-05-06 17:12:48 +02:00
|
|
|
$this->loginManager->generateStaySignedInToken($this->clientIpAddress);
|
2018-05-06 17:06:36 +02:00
|
|
|
|
|
|
|
$this->assertEquals(
|
2018-05-06 17:12:48 +02:00
|
|
|
sha1($this->passwordHash . $this->clientIpAddress . $this->salt),
|
2018-05-06 17:06:36 +02:00
|
|
|
$this->loginManager->getStaySignedInToken()
|
|
|
|
);
|
|
|
|
}
|
2018-05-06 17:12:48 +02:00
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user login - Shaarli has not yet been configured
|
|
|
|
*/
|
|
|
|
public function testCheckLoginStateNotConfigured()
|
|
|
|
{
|
|
|
|
$configManager = new \FakeConfigManager([
|
|
|
|
'resource.ban_file' => $this->banFile,
|
|
|
|
]);
|
|
|
|
$loginManager = new LoginManager($this->globals, $configManager, null);
|
|
|
|
$loginManager->checkLoginState([], '');
|
|
|
|
|
|
|
|
$this->assertFalse($loginManager->isLoggedIn());
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user login - the client cookie does not match the server token
|
|
|
|
*/
|
|
|
|
public function testCheckLoginStateStaySignedInWithInvalidToken()
|
|
|
|
{
|
2018-05-30 02:09:09 +02:00
|
|
|
// simulate a previous login
|
|
|
|
$this->session = [
|
|
|
|
'ip' => $this->clientIpAddress,
|
|
|
|
'expires_on' => time() + 100,
|
|
|
|
];
|
2018-05-06 17:12:48 +02:00
|
|
|
$this->loginManager->generateStaySignedInToken($this->clientIpAddress);
|
|
|
|
$this->cookie[LoginManager::$STAY_SIGNED_IN_COOKIE] = 'nope';
|
|
|
|
|
|
|
|
$this->loginManager->checkLoginState($this->cookie, $this->clientIpAddress);
|
|
|
|
|
2018-05-30 02:09:09 +02:00
|
|
|
$this->assertTrue($this->loginManager->isLoggedIn());
|
|
|
|
$this->assertTrue(empty($this->session['username']));
|
2018-05-06 17:12:48 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user login - the client cookie matches the server token
|
|
|
|
*/
|
|
|
|
public function testCheckLoginStateStaySignedInWithValidToken()
|
|
|
|
{
|
|
|
|
$this->loginManager->generateStaySignedInToken($this->clientIpAddress);
|
|
|
|
$this->cookie[LoginManager::$STAY_SIGNED_IN_COOKIE] = $this->loginManager->getStaySignedInToken();
|
|
|
|
|
|
|
|
$this->loginManager->checkLoginState($this->cookie, $this->clientIpAddress);
|
|
|
|
|
|
|
|
$this->assertTrue($this->loginManager->isLoggedIn());
|
2018-05-30 02:09:09 +02:00
|
|
|
$this->assertEquals($this->login, $this->session['username']);
|
|
|
|
$this->assertEquals($this->clientIpAddress, $this->session['ip']);
|
2018-05-06 17:12:48 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user login - the session has expired
|
|
|
|
*/
|
|
|
|
public function testCheckLoginStateSessionExpired()
|
|
|
|
{
|
|
|
|
$this->loginManager->generateStaySignedInToken($this->clientIpAddress);
|
|
|
|
$this->session['expires_on'] = time() - 100;
|
|
|
|
|
|
|
|
$this->loginManager->checkLoginState($this->cookie, $this->clientIpAddress);
|
|
|
|
|
|
|
|
$this->assertFalse($this->loginManager->isLoggedIn());
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user login - the remote client IP has changed
|
|
|
|
*/
|
|
|
|
public function testCheckLoginStateClientIpChanged()
|
|
|
|
{
|
|
|
|
$this->loginManager->generateStaySignedInToken($this->clientIpAddress);
|
|
|
|
|
|
|
|
$this->loginManager->checkLoginState($this->cookie, '10.7.157.98');
|
|
|
|
|
|
|
|
$this->assertFalse($this->loginManager->isLoggedIn());
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user credentials - wrong login supplied
|
|
|
|
*/
|
|
|
|
public function testCheckCredentialsWrongLogin()
|
|
|
|
{
|
|
|
|
$this->assertFalse(
|
|
|
|
$this->loginManager->checkCredentials('', '', 'b4dl0g1n', $this->password)
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user credentials - wrong password supplied
|
|
|
|
*/
|
|
|
|
public function testCheckCredentialsWrongPassword()
|
|
|
|
{
|
|
|
|
$this->assertFalse(
|
|
|
|
$this->loginManager->checkCredentials('', '', $this->login, 'b4dp455wd')
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user credentials - wrong login and password supplied
|
|
|
|
*/
|
|
|
|
public function testCheckCredentialsWrongLoginAndPassword()
|
|
|
|
{
|
|
|
|
$this->assertFalse(
|
|
|
|
$this->loginManager->checkCredentials('', '', 'b4dl0g1n', 'b4dp455wd')
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Check user credentials - correct login and password supplied
|
|
|
|
*/
|
|
|
|
public function testCheckCredentialsGoodLoginAndPassword()
|
|
|
|
{
|
|
|
|
$this->assertTrue(
|
|
|
|
$this->loginManager->checkCredentials('', '', $this->login, $this->password)
|
|
|
|
);
|
|
|
|
}
|
2017-10-25 23:03:31 +02:00
|
|
|
}
|