EventListener can't access wire-level request in case of failure

I’m trying to log the URI of failed requests using an EventListener. The URI may be rewritten by Interceptors and I’m interested in the URI of the wire-level request.

In case of a successful request, the URI of the wire-level request can be obtained by by calling Response.request(). But if the request fails and there is no Response object, there seems to be no way to get this information; Only the original is accessible through Call.request().

[swankjesse]

If you need a quick & dirty fix you can use Call tags and interceptors to hack around this. Here’s a sketch:

public void callOkHttp() {
  Request request = new Request.Builder()
      .url(...)
      .tag(HackWireLevelRequestHolder.class, new HackWireLevelRequestHolder())
      .build();
  ...
}


class HackWireLevelRequestHolder {
  public static final Interceptor INTERCEPTOR = new Interceptor() {
    public Response intercept(Chain chain) throws IOException {
      Request request = chain.request();
      request.tag(HackWireLevelRequestHolder.class).wireLevelRequest = request;
      return chain.proceed(request);
    }
  }

  Request wireLevelRequest = null;
}

[yschimke]

Just hit this but without the ability to control the creation of the request. Just realised I’m saved by the Call.Factory which I can trivially decorate to set the tag I need.

Same solution just using Call.Factory wrapper instead of an interceptor.

We might want to make Call.Factory usage best practices for clients? Coil, Media3, Retrofit all use it, so it’s standard anyway.