security icon indicating copy to clipboard operation
security copied to clipboard

Change parameter name $user in Nette\Security\Authenticator to $username

Open mildabre opened this issue 2 years ago • 0 comments

Hi,

interface Nette\Security\Authenticator has two string parameters: $user and $password. The name of the parameter $user may led to naming confusion in this cases:

a) with Nette\Security\User which is by default present in Nette presenters like $presenter->user and in Nette templates like $user

b) with the database table user - it is a common practice use table user with field username (login) for authentication

Therefore I propose to change the name of the parameter $user to $username like this:

interface Authenticator
{
	.......

	function authenticate(string $username, string $password): IIdentity;
}

github: https://github.com/nette/security/blob/0458647017a4b66cc6928c8c69ba86d66d5b2dc1/src/Security/Authenticator.php#L41

mildabre avatar May 18 '23 21:05 mildabre