fix: use JOIN instead of iterative queries

See #2402
ory · May 13, 2022 · 0998cfb · 0998cfb
1 parent 5277668
commit 0998cfb
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/persistence/sql/persister_identity.go b/persistence/sql/persister_identity.go
@@ -347,7 +347,7 @@ func (p *Persister) DeleteIdentity(ctx context.Context, id uuid.UUID) error {
 
 func (p *Persister) GetIdentity(ctx context.Context, id uuid.UUID) (*identity.Identity, error) {
 	var i identity.Identity
-	if err := p.GetConnection(ctx).Where("id = ? AND nid = ?", id, corp.ContextualizeNID(ctx, p.nid)).First(&i); err != nil {
+	if err := p.GetConnection(ctx).EagerPreload("VerifiableAddresses", "RecoveryAddresses").Where("id = ? AND nid = ?", id, corp.ContextualizeNID(ctx, p.nid)).First(&i); err != nil {
 		return nil, sqlcon.HandleError(err)
 	}