fix: handle user_code collisions

nsklikas · nsklikas · commit 8790af4f2c39 · 2025-01-09T09:32:58.000+01:00
diff --git a/persistence/sql/persister_device.go b/persistence/sql/persister_device.go
@@ -161,6 +161,8 @@ func (p *Persister) createDeviceAuthSession(ctx context.Context, deviceCodeSigna
 
 	if err = sqlcon.HandleError(p.CreateWithNetwork(ctx, req)); errors.Is(err, sqlcon.ErrConcurrentUpdate) {
 		return errors.Wrap(fosite.ErrSerializationFailure, err.Error())
+	} else if errors.Is(err, sqlcon.ErrUniqueViolation) {
+		return errors.Wrap(fosite.ErrExistingUserCodeSignature, err.Error())
 	} else if err != nil {
 		return err
 	}

Original file line number	Diff line number	Diff line change
`@@ -161,6 +161,8 @@ func (p *Persister) createDeviceAuthSession(ctx context.Context, deviceCodeSigna`
`161`	`161`
`162`	`162`	`if err = sqlcon.HandleError(p.CreateWithNetwork(ctx, req)); errors.Is(err, sqlcon.ErrConcurrentUpdate) {`
`163`	`163`	`return errors.Wrap(fosite.ErrSerializationFailure, err.Error())`
	`164`	`+ } else if errors.Is(err, sqlcon.ErrUniqueViolation) {`
	`165`	`+ return errors.Wrap(fosite.ErrExistingUserCodeSignature, err.Error())`
`164`	`166`	`} else if err != nil {`
`165`	`167`	`return err`
`166`	`168`	`}`