RUM-9503: Removing deprecated constructors

Author: satween

integrations/dd-sdk-android-okhttp/api/dd-sdk-android-okhttp.api

@@ -90,21 +90,6 @@ public abstract interface class com/datadog/android/okhttp/trace/TracedRequestLi
 }
 
 public class com/datadog/android/okhttp/trace/TracingInterceptor : okhttp3/Interceptor {
-	public fun <init> ()V
-	public fun <init> (Ljava/lang/String;)V
-	public fun <init> (Ljava/lang/String;Lcom/datadog/android/okhttp/trace/TracedRequestListener;)V
-	public fun <init> (Ljava/lang/String;Lcom/datadog/android/okhttp/trace/TracedRequestListener;Lcom/datadog/android/core/sampling/Sampler;)V
-	public synthetic fun <init> (Ljava/lang/String;Lcom/datadog/android/okhttp/trace/TracedRequestListener;Lcom/datadog/android/core/sampling/Sampler;ILkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public fun <init> (Ljava/lang/String;Ljava/util/List;)V
-	public fun <init> (Ljava/lang/String;Ljava/util/List;Lcom/datadog/android/okhttp/trace/TracedRequestListener;)V
-	public fun <init> (Ljava/lang/String;Ljava/util/List;Lcom/datadog/android/okhttp/trace/TracedRequestListener;Lcom/datadog/android/core/sampling/Sampler;)V
-	public synthetic fun <init> (Ljava/lang/String;Ljava/util/List;Lcom/datadog/android/okhttp/trace/TracedRequestListener;Lcom/datadog/android/core/sampling/Sampler;ILkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public fun <init> (Ljava/lang/String;Ljava/util/Map;)V
-	public fun <init> (Ljava/lang/String;Ljava/util/Map;Lcom/datadog/android/okhttp/trace/TracedRequestListener;)V
-	public fun <init> (Ljava/lang/String;Ljava/util/Map;Lcom/datadog/android/okhttp/trace/TracedRequestListener;Lcom/datadog/android/core/sampling/Sampler;)V
-	public synthetic fun <init> (Ljava/lang/String;Ljava/util/Map;Lcom/datadog/android/okhttp/trace/TracedRequestListener;Lcom/datadog/android/core/sampling/Sampler;ILkotlin/jvm/internal/DefaultConstructorMarker;)V
-	public fun <init> (Ljava/util/List;)V
-	public fun <init> (Ljava/util/Map;)V
 	public fun intercept (Lokhttp3/Interceptor$Chain;)Lokhttp3/Response;
 	protected fun onRequestIntercepted (Lcom/datadog/android/api/feature/FeatureSdkCore;Lokhttp3/Request;Lio/opentracing/Span;Lokhttp3/Response;Ljava/lang/Throwable;)V
 }

integrations/dd-sdk-android-okhttp/src/main/kotlin/com/datadog/android/okhttp/trace/TracingInterceptor.kt

@@ -104,125 +104,6 @@ internal constructor(
         onSdkInstanceReady(it as InternalSdkCore)
     }
 
-    /**
-     * Creates a [TracingInterceptor] to automatically create a trace around OkHttp [Request]s.
-     *
-     * @param sdkInstanceName SDK instance name to bind to, or null to check the default instance.
-     * Instrumentation won't be working until SDK instance is ready.
-     * @param tracedHosts a list of all the hosts that you want to be automatically tracked
-     * by this interceptor with Datadog style headers. If no host is provided (via this argument or global
-     * configuration [Configuration.Builder.setFirstPartyHosts]) the interceptor won't trace any OkHttp [Request],
-     * nor propagate tracing information to the backend.
-     * @param tracedRequestListener a listener for automatically created [Span]s
-     * @param traceSampler Sampler controlling the sampling of APM traces created for
-     * auto-instrumented requests. By default it is [DeterministicTraceSampler], which either can accept
-     * fixed sample rate or can get it dynamically from the provider. Value between `0.0` and
-     * `100.0`. A value of `0.0` means no trace will be kept, `100.0` means all traces will
-     * be kept (default value is `100.0`). If there is a parent trace attached to the network span created, then its
-     * sampling decision will be used instead.
-     */
-    @JvmOverloads
-    @Deprecated(
-        message = "This constructor is not going to be accessible anymore in future versions. " +
-            "Please use the Builder instead.",
-        replaceWith = ReplaceWith("TracingInterceptor.Builder(tracedHosts).build()")
-    )
-    constructor(
-        sdkInstanceName: String? = null,
-        tracedHosts: List<String>,
-        tracedRequestListener: TracedRequestListener = NoOpTracedRequestListener(),
-        traceSampler: Sampler<Span> = DeterministicTraceSampler(DEFAULT_TRACE_SAMPLE_RATE)
-    ) : this(
-        sdkInstanceName = sdkInstanceName,
-        tracedHosts = tracedHosts.associateWith {
-            setOf(
-                TracingHeaderType.DATADOG,
-                TracingHeaderType.TRACECONTEXT
-            )
-        },
-        tracedRequestListener = tracedRequestListener,
-        traceOrigin = null,
-        traceSampler = traceSampler,
-        traceContextInjection = TraceContextInjection.SAMPLED,
-        redacted404ResourceName = true,
-        localTracerFactory = DEFAULT_LOCAL_TRACER_FACTORY
-    )
-
-    /**
-     * Creates a [TracingInterceptor] to automatically create a trace around OkHttp [Request]s.
-     *
-     * @param sdkInstanceName SDK instance name to bind to, or null to check the default instance.
-     * Instrumentation won't be working until SDK instance is ready.
-     * @param tracedHostsWithHeaderType a list of all the hosts and header types that you want to be automatically tracked
-     * by this interceptor. If registering a GlobalTracer, the tracer must be configured with
-     * [AndroidTracer.Builder.setTracingHeaderTypes] containing all the necessary header types configured for OkHttp tracking.
-     * If no hosts are provided (via this argument or global
-     * configuration [Configuration.Builder.setFirstPartyHosts] or [Configuration.Builder.setFirstPartyHostsWithHeaderType] )
-     * the interceptor won't trace any OkHttp [Request], nor propagate tracing information to the backend.
-     * @param tracedRequestListener a listener for automatically created [Span]s
-     * @param traceSampler Sampler controlling the sampling of APM traces created for
-     * auto-instrumented requests. By default it is [DeterministicTraceSampler], which either can accept
-     * fixed sample rate or can get it dynamically from the provider. Value between `0.0` and
-     * `100.0`. A value of `0.0` means no trace will be kept, `100.0` means all traces will
-     * be kept (default value is `100.0`). If there is a parent trace attached to the network span created, then its
-     * sampling decision will be used instead.
-     */
-    @JvmOverloads
-    @Deprecated(
-        message = "This constructor is not going to be accessible anymore in future versions. " +
-            "Please use the Builder instead.",
-        replaceWith = ReplaceWith("TracingInterceptor.Builder(tracedHostsWithHeaderType).build()")
-    )
-    constructor(
-        sdkInstanceName: String? = null,
-        tracedHostsWithHeaderType: Map<String, Set<TracingHeaderType>>,
-        tracedRequestListener: TracedRequestListener = NoOpTracedRequestListener(),
-        traceSampler: Sampler<Span> = DeterministicTraceSampler(DEFAULT_TRACE_SAMPLE_RATE)
-    ) : this(
-        sdkInstanceName = sdkInstanceName,
-        tracedHosts = tracedHostsWithHeaderType,
-        tracedRequestListener = tracedRequestListener,
-        traceOrigin = null,
-        traceSampler = traceSampler,
-        traceContextInjection = TraceContextInjection.SAMPLED,
-        redacted404ResourceName = true,
-        localTracerFactory = DEFAULT_LOCAL_TRACER_FACTORY
-    )
-
-    /**
-     * Creates a [TracingInterceptor] to automatically create a trace around OkHttp [Request]s.
-     *
-     * @param sdkInstanceName SDK instance name to bind to, or null to check the default instance.
-     * Instrumentation won't be working until SDK instance is ready.
-     * @param tracedRequestListener a listener for automatically created [Span]s
-     * @param traceSampler Sampler controlling the sampling of APM traces created for
-     * auto-instrumented requests. By default it is [DeterministicTraceSampler], which either can accept
-     * fixed sample rate or can get it dynamically from the provider. Value between `0.0` and
-     * `100.0`. A value of `0.0` means no trace will be kept, `100.0` means all traces will
-     * be kept (default value is `100.0`). If there is a parent trace attached to the network span created, then its
-     * sampling decision will be used instead.
-     */
-    @JvmOverloads
-    @Deprecated(
-        message = "This constructor is not going to be accessible anymore in future versions. " +
-            "Please use the Builder instead.",
-        replaceWith = ReplaceWith("TracingInterceptor.Builder(tracedHosts).build()")
-    )
-    constructor(
-        sdkInstanceName: String? = null,
-        tracedRequestListener: TracedRequestListener = NoOpTracedRequestListener(),
-        traceSampler: Sampler<Span> = DeterministicTraceSampler(DEFAULT_TRACE_SAMPLE_RATE)
-    ) : this(
-        sdkInstanceName = sdkInstanceName,
-        tracedHosts = emptyMap(),
-        tracedRequestListener = tracedRequestListener,
-        traceOrigin = null,
-        traceSampler = traceSampler,
-        traceContextInjection = TraceContextInjection.SAMPLED,
-        redacted404ResourceName = true,
-        localTracerFactory = DEFAULT_LOCAL_TRACER_FACTORY
-    )
-
     init {
         val sdkCore = sdkCoreReference.get() as? FeatureSdkCore