Merge pull request #203 from 9seconds/simplify-time-randomization

Simplify TCP relay time randomization
9seconds · Aug 30, 2021 · fbc7499 · fbc7499
2 parents 00bb7be + ca5800c
commit fbc7499
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/mtglib/internal/relay/timeouts.go b/mtglib/internal/relay/timeouts.go
@@ -1,7 +1,6 @@
 package relay
 
 import (
-	"math"
 	"math/rand"
 	"time"
 )
@@ -15,12 +14,9 @@ func getTimeout() time.Duration {
 }
 
 func getTime(minDuration, maxDuration time.Duration) time.Duration {
-	minDurationInSeconds := minDuration.Seconds()
-	maxDurationInSeconds := maxDuration.Seconds()
-	middle := minDurationInSeconds + (maxDurationInSeconds-minDurationInSeconds)/2 // nolint: gomnd
-
-	number := minDurationInSeconds + rand.ExpFloat64()*middle
-	number = math.Round(math.Min(maxDurationInSeconds, number))
+	minDurationInSeconds := int(minDuration.Seconds())
+	maxDurationInSeconds := int(maxDuration.Seconds())
+	number := minDurationInSeconds + rand.Intn(maxDurationInSeconds-minDurationInSeconds)
 
 	return time.Duration(number) * time.Second
 }