digitalpoint
Well-known member
Is it intended that the table has two different unique indexes?
provider + provider_key definitely should be unique, but with user_id being a unique primary key, it means we can never have more than one external auth service per user because XenForo_Model_UserExternal->updateExternalAuthAssociation() does "ON DUPLICATE KEY UPDATE" if the user_id isn't unique even if the provider is different.
provider + provider_key definitely should be unique, but with user_id being a unique primary key, it means we can never have more than one external auth service per user because XenForo_Model_UserExternal->updateExternalAuthAssociation() does "ON DUPLICATE KEY UPDATE" if the user_id isn't unique even if the provider is different.