Fails to retry on 503 error about okhttp HOT 6 CLOSED

Just remember not to DoS your server! A 500 means it's having problems.

from okhttp.

OK, so two issues.

1. Your test is setting Retry-After on the Request, not on the MockResponse.
2. You are using a value of 5, which follows my earlier comment. You will need to implement this yourself. Only 0 is handled immediately.

from okhttp.

This is expected. You will need to perform retries in your app, or with an Interceptor.

from okhttp.

I understood that in the RetryAndFollowUpInterceptor class the 503 is meant to be retried at least once. So the test above was to check if it retried with 503 error. Please help me understand if I'm missing anything?

      HTTP_UNAVAILABLE -> {
        val priorResponse = userResponse.priorResponse
        if (priorResponse != null && priorResponse.code == HTTP_UNAVAILABLE) {
          // We attempted to retry and got another timeout. Give up.
          return null
        }

        if (retryAfter(userResponse, Integer.MAX_VALUE) == 0) {
          // specifically received an instruction to retry without delay
          return userResponse.request
        }

        return null
      }

from okhttp.

Damn, you are right. Sorry, bad memory on my part.

from okhttp.

Thanks! That makes sense now 👍

from okhttp.