OkHttp 3 IllegalStateException

Question

I customize an interceptor to refresh token when it's expired. When I get response body to detect if the token's expired, it throw java.lang.IllegalStateException: closed.

Here is my code with OkHttp 3

@Override
public Response intercept(Chain chain) throws IOException {
    Request original = chain.request();
    Request.Builder requestBuilder = original.newBuilder()
            .header("Content-Type", "application/json")
            .header(Constants.TAG_AUTHORIZATION, "Bearer " + token)
            .method(original.method(), original.body());


    Request request = requestBuilder.build();
    Response response = chain.proceed(request);
    if (response.code() == 200) {
        String json = response.body().string();
        try {
            JSONObject obj = new JSONObject(json);
            int code = obj.getInt(Constants.TAG_CODE);
            if (code == Constants.REQUEST_CODE_TOKEN_EXPIRED) {
                Response r = makeTokenRefreshCall(request, chain);
                return r;
            }
        } catch (JSONException e) {
            e.printStackTrace();
        }
    }
    return response;
}

Please help me!

@matrix the exception was not throw in intercept() but in my RxJava onError() method. — Hoang Lam, Jul 03 '17 at 08:35

score 4 · Answer 1 · answered Jul 03 '17 at 13:54

4

You're not allowed to consume the response body in an interceptor. If you'd like to work around this, use Response.peekBody() which makes a copy that you are allowed to read.

answered Jul 03 '17 at 13:54

Jesse Wilson

39,078
8
121
128

peekBody also throws IllegalStateException – IgorGanapolsky Nov 14 '17 at 17:29
ResponseBody responseBodyCopy = response.peekBody(Long.MAX_VALUE); responseBodyCopy.string(); – Veeresh Charantimath Feb 01 '19 at 08:25
I appreciate that this not only possible, but easy and without a large efficiency tradeoff. It would have been both easy and reasonable to omit in a library like this. Thank you @jesse-wilson – user1575326 Jan 30 '20 at 15:32

score 3 · Answer 2 · answered Jul 04 '17 at 08:22

3

Finally, I found a solution to read response body without exception

 ResponseBody body = response.body();
 BufferedSource source = body.source();
 if (source.request(1024)) throw new IOException("Body too long!");
 Buffer bufferedCopy = source.buffer().clone();
 ResponseBody newBody = ResponseBody.create(body.contentType(), body.contentLength(), bufferedCopy);
 String responseBodyString = newBody.string();

answered Jul 04 '17 at 08:22

Hoang Lam

523
8
23

This crashes with IllegalStateException – IgorGanapolsky Nov 14 '17 at 17:30

OkHttp 3 IllegalStateException

2 Answers2