Skip to content

fix: Check for provider ID in social organization signal

Summary

add_user_to_socialapp_organization was only checking for the provider string which leads to exclude Open ID providers or similar that relies on the provider ID. So no Open ID users are added to the org through the social app.

This MR update the queries to check for the provider ID string before falling back to the provider string

PS: This branch has mostly the same changes on the test side (besides the query number change to show the user assertion fail) to show that the test fails with an Open ID provider

Checklist

  • I am respecting Sentry's proprietary Business Source License (BSL) by implementing code in a clean room environment. I have not read any BSL Sentry code. It is OK to read MIT licensed code.
Edited by tytan652

Merge request reports

Loading