Apply jitter to maintenance check.

msimonides-proton · mateusz-markowicz · commit a9b9675395c8 · 2024-01-17T11:35:31.000+01:00
diff --git a/app/src/main/java/com/protonvpn/android/vpn/MaintenanceTracker.kt b/app/src/main/java/com/protonvpn/android/vpn/MaintenanceTracker.kt
@@ -24,15 +24,15 @@ class MaintenanceTracker(
         scope.launch {
             vpnStateMonitor.status.collect { status ->
                 if (status.state == VpnState.Connected) {
-                    task.scheduleIn(jitterMs(getScheduleDelay()))
+                    task.scheduleIn(getScheduleDelay())
                 } else if (status.state == VpnState.Disabled) {
                     task.cancelSchedule()
                 }
             }
         }
     }
 
-    private fun getScheduleDelay(): Long = TimeUnit.MINUTES.toMillis(appConfig.getMaintenanceTrackerDelay())
+    private fun getScheduleDelay(): Long = jitterMs(TimeUnit.MINUTES.toMillis(appConfig.getMaintenanceTrackerDelay()))
 
     private fun now() = SystemClock.elapsedRealtime()
 

Original file line number	Diff line number	Diff line change
`@@ -24,15 +24,15 @@ class MaintenanceTracker(`
`24`	`24`	`scope.launch {`
`25`	`25`	`vpnStateMonitor.status.collect { status ->`
`26`	`26`	`if (status.state == VpnState.Connected) {`
`27`		`- task.scheduleIn(jitterMs(getScheduleDelay()))`
	`27`	`+ task.scheduleIn(getScheduleDelay())`
`28`	`28`	`} else if (status.state == VpnState.Disabled) {`
`29`	`29`	`task.cancelSchedule()`
`30`	`30`	`}`
`31`	`31`	`}`
`32`	`32`	`}`
`33`	`33`	`}`
`34`	`34`
`35`		`- private fun getScheduleDelay(): Long = TimeUnit.MINUTES.toMillis(appConfig.getMaintenanceTrackerDelay())`
	`35`	`+ private fun getScheduleDelay(): Long = jitterMs(TimeUnit.MINUTES.toMillis(appConfig.getMaintenanceTrackerDelay()))`
`36`	`36`
`37`	`37`	`private fun now() = SystemClock.elapsedRealtime()`
`38`	`38`