OkHttp Interceptor che utilizza OkHttpClient senza ciclo di dipendenza

Question

Uso Retrofit e Dagger 2. Ho implementato un OkHttp Interceptor per aggiungere il token oauth. Nel caso in cui non vi sia alcun token oauth o il timestamp non è valido, richiedo uno nuovo (tramite il servizio Retrofit) prima che venga eseguita la richiesta effettiva.

Questo crea un ciclo di dipendenze in cui il servizio Retrofit richiede il Interceptor ma il Interceptor richiede anche il servizio Retrofit (per il recupero del token oauth).

Esempio per il Interceptor (per semplificazione, richiede sempre il token via restService#refreshAccessToken):

@Override 
public Response intercept(Chain chain) throws IOException { 
    Request originalRequest = chain.request(); 
    Request.Builder requestBuilder = originalRequest.newBuilder(); 

    String authHeader = "Bearer " + restService.refreshAccessToken(); 
    requestBuilder.addHeader("Authorization", authHeader); 
    return chain.proceed(requestBuilder.build()); 
} 

Answer 1

tuo Interceptor avrà bisogno di iniettare se stesso nel metodo Interceptor#intercept(). In questo modo il servizio di Retrofit (inclusa la dipendenza OkHttpClient con l'intercettatore aggiunto) può essere soddisfatto senza un ciclo di dipendenze.

Il mio NetworkComponent (che fornisce il mio servizio di retrofit) vive nella mia classe Application. Ecco un esempio di iniezione usando un'applicazione Context.

public class AuthInterceptor implements Interceptor { 

    @Inject RestService restService; 

    public AuthInterceptor(Context context) { 
     mContext = context; 
    } 

    @Override 
    public Response intercept(Chain chain) throws IOException { 
     // Inject Retrofit service 
     ((MyApplication) context.getApplicationContext()) 
       .getNetworkComponent().inject(this); 

     Request originalRequest = chain.request(); 
     Request.Builder requestBuilder = originalRequest.newBuilder(); 

     String authHeader = "Bearer " + restService.refreshAccessToken(); 
     requestBuilder.addHeader("Authorization", authHeader); 
     return chain.proceed(requestBuilder.build()); 
    } 
} 

È anche possibile impostare una variabile locale senza iniettare l'intera classe, ottenendo prestazioni migliori.

RestService restService = InjectHelper.getNetworkComponent(mContext).restService(); 

Answer 2

Io personalmente preferisco iniezione pigro, che è un po 'più pulito

public class AuthInterceptor implements Interceptor { 

    Lazy<RestService> restService; 

    public AuthInterceptor(Lazy<RestService> restService) { 
     this.restService = restService; 
    } 

    @Override 
    public Response intercept(Chain chain) throws IOException { 
     Request originalRequest = chain.request(); 
     Request.Builder requestBuilder = originalRequest.newBuilder(); 

     String authHeader = "Bearer " + restService.get().refreshAccessToken(); 
     requestBuilder.addHeader("Authorization", authHeader); 
     return chain.proceed(requestBuilder.build()); 
    } 
}