Avoid override of id/pk attributes with service provided data. Closes #217

diff --git a/social_auth/backends/pipeline/user.py b/social_auth/backends/pipeline/user.py
index b1acc48824565e4aae36ae68e837631a452e35d8..eaca1ae261658655235e3c7f440e0ab04ba779b4 100644 (file)
--- a/social_auth/backends/pipeline/user.py
+++ b/social_auth/backends/pipeline/user.py
@@ -92,7 +92,7 @@ def update_user_details(backend, details, response, user, is_new=False, *args,
     if not getattr(settings, 'SOCIAL_AUTH_CHANGE_SIGNAL_ONLY', False):
         for name, value in details.iteritems():
             # do not update username, it was already generated
-            if name == USERNAME:
+            if name in (USERNAME, 'id', 'pk'):
                 continue
             if value and value != getattr(user, name, None):
                 setattr(user, name, value)