trmnl/trmnl-android
0
0
Fork 0
mirror of https://github.com/usetrmnl/trmnl-android.git synced 2026-04-29 13:35:26 -07:00
Code Issues Packages Projects Releases Wiki Activity

Refactor: Use existing HTTP_429 constant instead of defining new one

Browse Source 
- Import HTTP_429 from ink.trmnl.android.util package
- Remove duplicate HTTP_TOO_MANY_REQUESTS constant
- Maintains consistency with rest of codebase
- All tests pass
This commit is contained in:
Hossain Khan
2026-02-08 19:31:51 -05:00
parent 5aa47c689f
commit 5ce06a728a
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/src/main/java/ink/trmnl/android/network/RateLimitInterceptor.kt
+3 -7
View File
@@ -1,5 +1,6 @@
package ink.trmnl.android.network
import ink.trmnl.android.util.HTTP_429
import okhttp3.Interceptor
import okhttp3.Response
import timber.log.Timber
@@ -46,11 +47,6 @@ class RateLimitInterceptor : Interceptor {
* Prevents exponential backoff from growing too large.
*/
private const val MAX_BACKOFF_MS = 32_000L
/**
* HTTP status code for "Too Many Requests" (rate limiting).
*/
private const val HTTP_TOO_MANY_REQUESTS = 429
}
override fun intercept(chain: Interceptor.Chain): Response {
@@ -59,7 +55,7 @@ class RateLimitInterceptor : Interceptor {
var attempt = 0
// Retry loop for handling 429 responses
while (response.code == HTTP_TOO_MANY_REQUESTS && attempt < MAX_RETRIES) {
while (response.code == HTTP_429 && attempt < MAX_RETRIES) {
attempt++
// Calculate backoff delay
@@ -86,7 +82,7 @@ class RateLimitInterceptor : Interceptor {
}
// Log if we exhausted all retries
if (response.code == HTTP_TOO_MANY_REQUESTS && attempt >= MAX_RETRIES) {
if (response.code == HTTP_429 && attempt >= MAX_RETRIES) {
Timber.tag(TAG).e(
"Rate limit exceeded (HTTP 429) for ${request.url}. " +
"Exhausted all $MAX_RETRIES retry attempts. Giving up.",