updateCredentials usually works together with extractCredentials. extractCredentials usually does not know user ids - they become only known after authentication. Therefore, the login parameter of updateCredentials must refer to the user name (which actually is the login name) and not the user id -- as the name login suggests.
In #105 @rpatterson reported a violation of this condition in events.userCredentialsUpdatedHandler.
This PR fixes #105.
It also fixes the docstring of PluggableAuthService._extractUserIds.
updateCredentialsusually works together withextractCredentials.extractCredentialsusually does not know user ids - they become only known after authentication. Therefore, theloginparameter ofupdateCredentialsmust refer to the user name (which actually is the login name) and not the user id -- as the nameloginsuggests. In #105 @rpatterson reported a violation of this condition inevents.userCredentialsUpdatedHandler.This PR fixes #105. It also fixes the docstring of
PluggableAuthService._extractUserIds.