tidy: double check generated connection IDs are unique

webmeshproj · Nov 11, 2023 · c7a3618 · c7a3618
1 parent 3ba1f2e
commit c7a3618
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/cmd/daemoncmd/server.go b/pkg/cmd/daemoncmd/server.go
@@ -172,6 +172,10 @@ func (app *AppDaemon) PutConnection(ctx context.Context, req *v1.PutConnectionRe
 			return nil, status.Errorf(codes.Internal, "failed to generate connection ID: %v", err)
 		}
 		profileID = ProfileID(id)
+		// Double check that the generated ID is unique
+		if _, err := app.connmgr.Profiles().Get(ctx, profileID); err == nil {
+			return nil, status.Errorf(codes.Internal, "generated connection ID is not unique: %v", profileID)
+		}
 	}
 	err = app.connmgr.Profiles().Put(ctx, profileID, Profile{req})
 	if err != nil {