Skip to content

Commit 1a902c0

Browse files
fix: when skipping a call due to throttle count one more skipped (#100)
1 parent 5c52d66 commit 1a902c0

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

unleashandroidsdk/src/main/java/io/getunleash/android/polling/UnleashFetcher.kt

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -95,15 +95,7 @@ open class UnleashFetcher(
9595
}
9696

9797
suspend fun refreshToggles(): ToggleResponse {
98-
if (throttler.performAction()) {
99-
Log.d(TAG, "Refreshing toggles")
100-
val response = doFetchToggles(unleashContext.value)
101-
fetcherHeartbeatFlow.emit(HeartbeatEvent(response.status, response.error?.message))
102-
return response
103-
}
104-
Log.i(TAG, "Skipping refresh toggles due to throttling")
105-
fetcherHeartbeatFlow.emit(HeartbeatEvent(Status.THROTTLED))
106-
return ToggleResponse(Status.THROTTLED)
98+
return this.refreshTogglesWithContext(unleashContext.value);
10799
}
108100

109101
suspend fun refreshTogglesWithContext(ctx: UnleashContext): ToggleResponse {
@@ -113,6 +105,7 @@ open class UnleashFetcher(
113105
fetcherHeartbeatFlow.emit(HeartbeatEvent(response.status, response.error?.message))
114106
return response
115107
}
108+
throttler.skipped() // count skipped requests
116109
Log.i(TAG, "Skipping refresh toggles due to throttling")
117110
fetcherHeartbeatFlow.emit(HeartbeatEvent(Status.THROTTLED))
118111
return ToggleResponse(Status.THROTTLED)

0 commit comments

Comments
 (0)