New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possibility to login by email #12543
Conversation
plugins/Login/Controller.php
Outdated
@@ -118,6 +121,19 @@ function login($messageNoAccess = null, $infoMessage = false) | |||
return $view->render(); | |||
} | |||
|
|||
protected function getLoginFromLoginOrEmail($loginOrEmail) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Be good to have this method ideally outside the controller or make it at least private so it can for sure not be executed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed the method to private
Does "lost your password" work with email? Haven't tested. |
Tested successful logins / and failed logins: Works and no information is exposed on unsuccessful logins etc 👍 |
Password resetting already worked with email. See https://github.com/matomo-org/matomo/blob/3.x-dev/plugins/Login/PasswordResetter.php#L161-L176 |
This is quite simple solution for #11959.