Authentication methods refactor #276
Description
Currently the Ldap auth implementation syncs ldap user and group info to the mms db - this is because the login process actually queries the db to get group (granted authority) info even though the authentication object should already have 'granted authority' info. The intended behavior is for the db to only include 'local' relationships between user and groups, and the authentication token to include the union of the local groups + third party (ldap) groups.
- this should not be added https://github.com/Open-MBEE/exec-mms/blob/develop/ldap/src/main/java/org/openmbee/mms/ldap/LdapSecurityConfig.java#L210
- the user details here is relying only on the db to generate token, the token should really be combining granted authorities from the Authentication class with what's in the db https://github.com/Open-MBEE/exec-mms/blob/4.0.20/authenticator/src/main/java/org/openmbee/mms/authenticator/controllers/AuthenticationController.java#L48
- check other authentications for similar traps