JakeHillion/storj
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

satellite/console: proper constraint error check for duplicate project members

Browse Source 
The string check previously used to check for constraint errors is now
replaced with dbx.IsConstraintError check.

Change-Id: I553ccd69e3c02b6b54441bd9f929b85a155eaf00
This commit is contained in:
Wilfred Asomani 2023-05-08 11:09:02 +00:00 committed by Storj Robot
parent 018b6eeeaf
commit 5145be1eeb
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
satellite/console/service.go
View File

@ -37,6 +37,7 @@ import (
"storj.io/storj/satellite/mailservice"
"storj.io/storj/satellite/payments"
"storj.io/storj/satellite/payments/billing"
"storj.io/storj/satellite/satellitedb/dbx"
)
var mon = monkit.Package()
@ -1965,7 +1966,7 @@ func (s *Service) AddProjectMembers(ctx context.Context, projectID uuid.UUID, em
err = s.store.WithTx(ctx, func(ctx context.Context, tx DBTx) error {
for _, user := range users {
if _, err := tx.ProjectMembers().Insert(ctx, user.ID, isMember.project.ID); err != nil {
if strings.Contains(err.Error(), "duplicate key") {
if dbx.IsConstraintError(err) {
return errs.New("%s is already on the project", user.Email)
}
return err