Merge pull request #14 from rb3ckers/fix-range-bug

fix(targets): Fix range bug
rb3ckers · Jan 22, 2020 · e276819 · e276819
2 parents 7405f3d + 34a2d3b
commit e276819
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/datatypes/MirrorTargets.go b/datatypes/MirrorTargets.go
@@ -72,9 +72,9 @@ func (mt *MirrorTargets) ForEach(f func(string, *gobreaker.CircuitBreaker)) {
 	}
 }
 
-func (mt *MirrorTargets) ListTargets() []Target {
-	targets := make([]Target, len(mt.targets)-1)
-
+func (mt *MirrorTargets) ListTargets() []*Target {
+	targets := make([]*Target, len(mt.targets))
+	i := 0
 	for url, target := range mt.targets {
 		var state string
 		switch target.circuitBreaker.State() {
@@ -88,11 +88,12 @@ func (mt *MirrorTargets) ListTargets() []Target {
 			state = "unknown"
 		}
 
-		targets = append(targets, Target{
+		targets[i] = &Target{
 			Name:         url,
 			FailingSince: target.firstFailure,
 			State:        state,
-		})
+		}
+		i = i + 1
 	}
 	return targets
 }