-
Notifications
You must be signed in to change notification settings - Fork 13
Open
Labels
Priority: MediumThis issue may be useful, and needs some attention.This issue may be useful, and needs some attention.help wantedExtra attention is neededExtra attention is neededrefactoringPull requests that refactor code but do not change its behavior.Pull requests that refactor code but do not change its behavior.
Milestone
Description
During development of #1125 I found that the definition of the authentication methods in user.py is split into the attributes
authenticationProviders
secondFactorProviders
validAuthenticationMethods
but at least, the only true definition should be just in validAuthenticationMethods
; Why should I register an auth_userpassword
-login, when validAuthenticationMethods
does only define GoogleLogin?
This is a relict and bad extension of old code base, and must be refactored.
Metadata
Metadata
Assignees
Labels
Priority: MediumThis issue may be useful, and needs some attention.This issue may be useful, and needs some attention.help wantedExtra attention is neededExtra attention is neededrefactoringPull requests that refactor code but do not change its behavior.Pull requests that refactor code but do not change its behavior.