UserPanel: Better design of "No identity."

.travis.yml

@@ -27,25 +27,29 @@ jobs:
 
 
         -   name: Nette Code Checker
+            php: 7.4
             install:
                 - travis_retry composer create-project nette/code-checker temp/code-checker ^3 --no-progress
             script:
                 - php temp/code-checker/code-checker --strict-types
 
 
         -   name: Nette Coding Standard
+            php: 7.4
             install:
                 - travis_retry composer create-project nette/coding-standard temp/coding-standard ^2 --no-progress
             script:
                 - php temp/coding-standard/ecs check src tests --config temp/coding-standard/coding-standard-php71.yml
 
 
         -   stage: Static Analysis (informative)
+            php: 7.4
             script:
                 - composer run-script phpstan
 
 
         -   stage: Code Coverage
+            php: 7.4
             script:
                 - vendor/bin/tester -p phpdbg tests -s --coverage ./coverage.xml --coverage-src ./src
             after_script:

src/Bridges/SecurityDI/SecurityExtension.php

@@ -35,9 +35,10 @@ public function getConfigSchema(): Nette\Schema\Schema
 			'users' => Expect::arrayOf(
 				Expect::anyOf(
 					Expect::string(), // user => password
-					Expect::structure([ // user => password + roles
+					Expect::structure([ // user => password + roles + data
 						'password' => Expect::string(),
 						'roles' => Expect::anyOf(Expect::string(), Expect::listOf('string')),
+						'data' => Expect::array(),
 					])->castTo('array')
 				)
 			),
@@ -49,6 +50,7 @@ public function getConfigSchema(): Nette\Schema\Schema
 
 	public function loadConfiguration()
 	{
+		/** @var object{debugger: bool, users: array, roles: array, resources: array} $config */
 		$config = $this->config;
 		$builder = $this->getContainerBuilder();
 
@@ -69,17 +71,18 @@ public function loadConfiguration()
 		}
 
 		if ($config->users) {
-			$usersList = $usersRoles = [];
+			$usersList = $usersRoles = $usersData = [];
 			foreach ($config->users as $username => $data) {
 				$data = is_array($data) ? $data : ['password' => $data];
-				$this->validateConfig(['password' => null, 'roles' => null], $data, $this->prefix("security.users.$username"));
+				$this->validateConfig(['password' => null, 'roles' => null, 'data' => []], $data, $this->prefix("security.users.$username"));
 				$usersList[$username] = $data['password'];
 				$usersRoles[$username] = $data['roles'] ?? null;
+				$usersData[$username] = $data['data'] ?? [];
 			}
 
 			$builder->addDefinition($this->prefix('authenticator'))
 				->setType(Nette\Security\IAuthenticator::class)
-				->setFactory(Nette\Security\SimpleAuthenticator::class, [$usersList, $usersRoles]);
+				->setFactory(Nette\Security\SimpleAuthenticator::class, [$usersList, $usersRoles, $usersData]);
 
 			if ($this->name === 'security') {
 				$builder->addAlias('nette.authenticator', $this->prefix('authenticator'));

src/Bridges/SecurityTracy/templates/UserPanel.panel.phtml

@@ -9,5 +9,5 @@ use Tracy\Dumper;
 <div class="nette-UserPanel">
 	<h1><?php if ($user->isLoggedIn()): ?>Logged in<?php else: ?>Unlogged<?php endif ?></h1>
 
-	<?php if ($user->getIdentity()): echo Dumper::toHtml($user->getIdentity(), [Dumper::LIVE => true]); else: ?><p>no identity</p><?php endif ?>
+	<?php if ($user->getIdentity()): echo Dumper::toHtml($user->getIdentity(), [Dumper::LIVE => true]); else: ?><p style="text-align:center;padding:1em 0;color:#888">No&nbsp;identity.</p><?php endif ?>
 </div>

src/Security/SimpleAuthenticator.php

@@ -25,15 +25,20 @@ class SimpleAuthenticator implements IAuthenticator
 	/** @var array */
 	private $usersRoles;
 
+	/** @var array */
+	private $usersData;
+
 
 	/**
 	 * @param  array  $userlist list of pairs username => password
 	 * @param  array  $usersRoles list of pairs username => role[]
+	 * @param  array  $usersData list of pairs username => mixed[]
 	 */
-	public function __construct(array $userlist, array $usersRoles = [])
+	public function __construct(array $userlist, array $usersRoles = [], array $usersData = [])
 	{
 		$this->userlist = $userlist;
 		$this->usersRoles = $usersRoles;
+		$this->usersData = $usersData;
 	}
 
 
@@ -48,7 +53,7 @@ public function authenticate(array $credentials): IIdentity
 		foreach ($this->userlist as $name => $pass) {
 			if (strcasecmp($name, $username) === 0) {
 				if ((string) $pass === (string) $password) {
-					return new Identity($name, $this->usersRoles[$name] ?? null);
+					return new Identity($name, $this->usersRoles[$name] ?? null, $this->usersData[$name] ?? []);
 				} else {
 					throw new AuthenticationException('Invalid password.', self::INVALID_CREDENTIAL);
 				}

tests/Security/SimpleAuthenticator.Data.phpt

@@ -0,0 +1,37 @@
+<?php
+
+/**
+ * Test: Nette\Security\SimpleAuthenticator and data
+ */
+
+declare(strict_types=1);
+
+use Nette\Security\SimpleAuthenticator;
+use Tester\Assert;
+
+
+require __DIR__ . '/../bootstrap.php';
+
+
+$users = [
+	'john' => 'john123',
+	'admin' => 'admin123',
+	'user' => 'user123',
+];
+$usersData = [
+	'admin' => ['nick' => 'admin', 'email' => '[email protected]'],
+	'user' => ['nick' => 'user', 'email' => '[email protected]'],
+];
+$expectedData = [
+	'admin' => ['nick' => 'admin', 'email' => '[email protected]'],
+	'user' => ['nick' => 'user', 'email' => '[email protected]'],
+	'john' => [],
+];
+
+$authenticator = new SimpleAuthenticator($users, [], $usersData);
+
+foreach ($users as $username => $password) {
+	$identity = $authenticator->authenticate([$username, $password]);
+	Assert::equal($username, $identity->getId());
+	Assert::equal($expectedData[$username], $identity->getData());
+}