Avoid naming csprg when cryptographic security is not required

Author: tim-barry

cmd/bootstrap/cmd/rootblock.go

@@ -249,12 +249,12 @@ func rootBlock(cmd *cobra.Command, args []string) {
 	if err != nil {
 		log.Fatal().Err(err).Msg("unable to generate random seed")
 	}
-	csprg, err := prg.New(randomSeed[:], prg.BootstrapClusterAssignment, nil)
+	clusteringPrg, err := prg.New(randomSeed[:], prg.BootstrapClusterAssignment, nil)
 	if err != nil {
 		log.Fatal().Err(err).Msg("unable to initialize pseudorandom generator")
 	}
 	log.Info().Msg("computing collection node clusters")
-	assignments, clusters, err := common.ConstructClusterAssignment(log, model.ToIdentityList(partnerNodes), model.ToIdentityList(internalNodes), int(flagCollectionClusters), csprg)
+	assignments, clusters, err := common.ConstructClusterAssignment(log, model.ToIdentityList(partnerNodes), model.ToIdentityList(internalNodes), int(flagCollectionClusters), clusteringPrg)
 	if err != nil {
 		log.Fatal().Err(err).Msg("unable to generate cluster assignment")
 	}

cmd/bootstrap/run/epochs.go

@@ -176,12 +176,12 @@ func GenerateRecoverTxArgsWithDKG(
 		}
 	}
 
-	csprg, err := prg.New(epoch.RandomSource(), prg.BootstrapClusterAssignment, nil)
+	rng, err := prg.New(epoch.RandomSource(), prg.BootstrapClusterAssignment, nil)
 	if err != nil {
 		return nil, fmt.Errorf("could not initialize PRNG: %w", err)
 	}
 	log.Info().Msgf("partitioning %d partners + %d internal nodes into %d collector clusters", len(partnerCollectors), len(internalCollectors), collectionClusters)
-	assignments, clusters, err := common.ConstructClusterAssignment(log, partnerCollectors, internalCollectors, collectionClusters, csprg)
+	assignments, clusters, err := common.ConstructClusterAssignment(log, partnerCollectors, internalCollectors, collectionClusters, rng)
 	if err != nil {
 		return nil, fmt.Errorf("unable to generate cluster assignment: %w", err)
 	}