fix: round bandwidth stats (#3735)

shikaan · web-flow · commit 58fb802a05f7 · 2021-07-27T10:26:01.000+01:00
Dividing the bandwidth stats sometimes results in a fraction which we cannot convert to a `BigInteger` since it's not an integer... Fixes #3726
diff --git a/packages/ipfs-core/src/components/stats/bw.js b/packages/ipfs-core/src/components/stats/bw.js
@@ -53,10 +53,10 @@ function getBandwidthStats (libp2p, opts) {
   const { movingAverages, snapshot } = stats
 
   return {
-    totalIn: BigInt(snapshot.dataReceived.toString()),
-    totalOut: BigInt(snapshot.dataSent.toString()),
-    rateIn: BigInt(movingAverages.dataReceived[60000].movingAverage() / 60),
-    rateOut: BigInt(movingAverages.dataSent[60000].movingAverage() / 60)
+    totalIn: BigInt(snapshot.dataReceived.integerValue().toString()),
+    totalOut: BigInt(snapshot.dataSent.integerValue().toString()),
+    rateIn: BigInt(Math.round(movingAverages.dataReceived[60000].movingAverage() / 60)),
+    rateOut: BigInt(Math.round(movingAverages.dataSent[60000].movingAverage() / 60))
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -53,10 +53,10 @@ function getBandwidthStats (libp2p, opts) {`
`53`	`53`	`const { movingAverages, snapshot } = stats`
`54`	`54`
`55`	`55`	`return {`
`56`		`- totalIn: BigInt(snapshot.dataReceived.toString()),`
`57`		`- totalOut: BigInt(snapshot.dataSent.toString()),`
`58`		`- rateIn: BigInt(movingAverages.dataReceived[60000].movingAverage() / 60),`
`59`		`- rateOut: BigInt(movingAverages.dataSent[60000].movingAverage() / 60)`
	`56`	`+ totalIn: BigInt(snapshot.dataReceived.integerValue().toString()),`
	`57`	`+ totalOut: BigInt(snapshot.dataSent.integerValue().toString()),`
	`58`	`+ rateIn: BigInt(Math.round(movingAverages.dataReceived[60000].movingAverage() / 60)),`
	`59`	`+ rateOut: BigInt(Math.round(movingAverages.dataSent[60000].movingAverage() / 60))`
`60`	`60`	`}`
`61`	`61`	`}`
`62`	`62`