Fixes 500 when user info fetched from Keycloak

[knikolla]

When creating a users, we first attempt to fetch up to date information from Keycloak. However, when that succeeds, the API returns 500 because the dictionary returned contains an extra key.

TypeError: create_user() got an unexpected keyword argument 'source'.

This fixes the above error by passing only the required keys to create_user.

[joachimweyl]

Is this ready for Review?

[QuanMPhm]

The changes look fine. The only question seems to be whether it makes sense to check if user data obtained from Keycloak has the minimum required data, and whether we should sanitize them or not, similar to how register_users_from_csv.py sanitizes user data prior to user registration.

[knikolla]

The changes look fine. The only question seems to be whether it makes sense to check if user data obtained from Keycloak has the minimum required data, and whether we should sanitize them or not, similar to how register_users_from_csv.py sanitizes user data prior to user registration.

The only sanitization happening from the user registration script is if a RoleBinding on OpenShift needs to be created (that is done external to ColdFront and is applied directly to the OpenShift cluster). We don't do any form of sanitization for the communication to ColdFront itself.