get_user_details() vs user_data() about python-social-auth

[omab]

@gsakkis, get_user_details() should be defined on the backend that works with django-auth mechanism, while user_data() belongs to the authentication class used by DSA. In your case user_data() should call that service and retrieve the needed user information, then on get_user_details() you should transform that data into the expected format for DSA which is:

{
    'username': <username if any>,
    'email': <user email if any>,
    'fullname': <user full name if any>,
    'first_name': <user first name if any>,
    'last_name': <user last name if any>
}

[gsakkis]

Ah, so the response parameter of get_user_details() is the result of user_data(); I thought they were called in the opposite order.

Pull request coming soon, thanks!

[omab]

@gsakkis, when I wrote my previous reply I thought I was on django-social-auth project, not python-social-auth, it still applies (kinda), so let me rewrite it.

user_data() should call the service that provide the extra user information, then get_user_details() must transform it to into the format that PSA expect, which is:

{
    'username': <username if any>,
    'email': <user email if any>,
    'fullname': <user full name if any>,
    'first_name': <user first name if any>,
    'last_name': <user last name if any>
}

[omab]

I think we can close this one.