impl: let upstream code decide which interceptors to install

Proxy support is enabled for every upstream usage

Author: fioan89

src/main/kotlin/com/coder/toolbox/sdk/CoderHttpClientBuilder.kt

@@ -1,22 +1,19 @@
 package com.coder.toolbox.sdk
 
 import com.coder.toolbox.CoderToolboxContext
-import com.coder.toolbox.sdk.interceptors.Interceptors
 import com.coder.toolbox.util.CoderHostnameVerifier
 import com.coder.toolbox.util.coderSocketFactory
 import com.coder.toolbox.util.coderTrustManagers
 import com.jetbrains.toolbox.api.remoteDev.connection.ProxyAuth
 import okhttp3.Credentials
+import okhttp3.Interceptor
 import okhttp3.OkHttpClient
-import java.net.URL
 import javax.net.ssl.X509TrustManager
 
 object CoderHttpClientBuilder {
     fun build(
         context: CoderToolboxContext,
-        pluginVersion: String,
-        url: URL,
-        token: String?,
+        interceptors: List<Interceptor>
     ): OkHttpClient {
         val settings = context.settingsStore.readOnly()
 
@@ -46,20 +43,14 @@ object CoderHttpClientBuilder {
                 .build()
         }
 
-        if (context.settingsStore.requireTokenAuth) {
-            if (token.isNullOrBlank()) {
-                throw IllegalStateException("Token is required for $url deployment")
-            }
-            builder = builder.addInterceptor(Interceptors.tokenAuth(token))
-        }
-
-        return builder
-            .sslSocketFactory(socketFactory, trustManagers[0] as X509TrustManager)
+        builder.sslSocketFactory(socketFactory, trustManagers[0] as X509TrustManager)
             .hostnameVerifier(CoderHostnameVerifier(settings.tls.altHostname))
             .retryOnConnectionFailure(true)
-            .addInterceptor(Interceptors.userAgent(pluginVersion))
-            .addInterceptor(Interceptors.externalHeaders(context, url))
-            .addInterceptor(Interceptors.logging(context))
-            .build()
+
+        interceptors.forEach { interceptor ->
+            builder.addInterceptor(interceptor)
+
+        }
+        return builder.build()
     }
 }

src/main/kotlin/com/coder/toolbox/sdk/CoderRestClient.kt

@@ -7,6 +7,7 @@ import com.coder.toolbox.sdk.convertors.LoggingConverterFactory
 import com.coder.toolbox.sdk.convertors.OSConverter
 import com.coder.toolbox.sdk.convertors.UUIDConverter
 import com.coder.toolbox.sdk.ex.APIResponseException
+import com.coder.toolbox.sdk.interceptors.Interceptors
 import com.coder.toolbox.sdk.v2.CoderV2RestFacade
 import com.coder.toolbox.sdk.v2.models.ApiErrorResponse
 import com.coder.toolbox.sdk.v2.models.BuildInfo
@@ -59,12 +60,21 @@ open class CoderRestClient(
                 .add(OSConverter())
                 .add(UUIDConverter())
                 .build()
+        val interceptors = buildList {
+            if (context.settingsStore.requireTokenAuth) {
+                if (token.isNullOrBlank()) {
+                    throw IllegalStateException("Token is required for $url deployment")
+                }
+                add(Interceptors.tokenAuth(token))
+            }
+            add((Interceptors.userAgent(pluginVersion)))
+            add(Interceptors.externalHeaders(context, url))
+            add(Interceptors.logging(context))
+        }
 
         httpClient = CoderHttpClientBuilder.build(
             context,
-            pluginVersion,
-            url,
-            token
+            interceptors
         )
 
         retroRestClient =