fix: remove redundant ORDER BY in QueryForCredentials

ory-bot · ory-bot · commit 65b27fd812b2 · 2026-03-04T13:07:08.000Z
GitOrigin-RevId: 78e60dcdf2ed5082ffa3b7e5489ab9018fc2f5b2
diff --git a/persistence/sql/identity/persister_identity.go b/persistence/sql/identity/persister_identity.go
@@ -879,8 +879,6 @@ func QueryForCredentials(con *pop.Connection, where ...Where) (credentialsPerIde
 		"identity_credentials.updated_at",
 	).LeftJoin(identifiersTableNameWithIndexHint(con),
 		"identity_credential_identifiers.identity_credential_id = identity_credentials.id AND identity_credential_identifiers.nid = identity_credentials.nid",
-	).Order(
-		"identity_credential_identifiers.identifier ASC",
 	)
 	for _, w := range where {
 		q = q.Where("("+w.Condition+")", w.Args...)

Original file line number	Diff line number	Diff line change
`@@ -879,8 +879,6 @@ func QueryForCredentials(con *pop.Connection, where ...Where) (credentialsPerIde`
`879`	`879`	`"identity_credentials.updated_at",`
`880`	`880`	`).LeftJoin(identifiersTableNameWithIndexHint(con),`
`881`	`881`	`"identity_credential_identifiers.identity_credential_id = identity_credentials.id AND identity_credential_identifiers.nid = identity_credentials.nid",`
`882`		`- ).Order(`
`883`		`- "identity_credential_identifiers.identifier ASC",`
`884`	`882`	`)`
`885`	`883`	`for _, w := range where {`
`886`	`884`	`q = q.Where("("+w.Condition+")", w.Args...)`