lint

.golangci.yml

@@ -25,7 +25,6 @@ linters:
     - typecheck
     - tenv
     - unconvert
-    # Prefer unparam over revive's unused param. It is more thorough in its checking.
     - unparam
     - unused
     - misspell

broadcast.go

@@ -2,10 +2,11 @@ package main
 
 import (
 	"context"
+	"crypto/rand"
 	"encoding/hex"
 	"fmt"
 	"log"
-	"math/rand"
+	"math/big"
 	"net/http"
 	"strings"
 	"time"
@@ -179,16 +180,18 @@ func BroadcastTransaction(txBytes []byte, rpcEndpoint string) (*coretypes.Result
 }
 
 func generateRandomString(config Config) (string, error) {
-	src := rand.NewSource(time.Now().UnixNano())
-	r := rand.New(src)
-
-	sizeB := r.Intn(config.RandMax-config.RandMin+1) + config.RandMin // Generate random size between 300000 and 400000 bytes
+	// Generate a random size between config.RandMin and config.RandMax
+	sizeB, err := rand.Int(rand.Reader, big.NewInt(int64(config.RandMax-config.RandMin+1)))
+	if err != nil {
+		return "", err
+	}
+	sizeB = sizeB.Add(sizeB, big.NewInt(int64(config.RandMin)))
 
 	// Calculate the number of bytes to generate (2 characters per byte in hex encoding)
-	nBytes := sizeB / 2
+	nBytes := int(sizeB.Int64()) / 2
 
 	randomBytes := make([]byte, nBytes)
-	_, err := r.Read(randomBytes)
+	_, err = rand.Read(randomBytes)
 	if err != nil {
 		return "", err
 	}

main.go

@@ -70,7 +70,7 @@ func main() {
 			sequenceMap[nodeURL] = 0
 			sequenceMu.Unlock()
 
-			for {
+			for { //nolint:gosimple // need to fis this but I keep finding solutinos that don't do the same thing as this.
 				select {
 				case <-transactionCh:
 					sequenceMu.Lock()