Remove deprecation warning for httpLoggingInterceptor config property.

Author: marcin-cebo

pubnub-gson/pubnub-gson-api/src/main/java/com/pubnub/api/java/v2/PNConfiguration.kt

@@ -252,14 +252,10 @@ interface PNConfiguration : com.pubnub.api.v2.PNConfiguration {
         val certificatePinner: CertificatePinner?
 
         /**
-         * Sets a custom [HttpLoggingInterceptor] for logging network traffic.
+         * Sets a custom [HttpLoggingInterceptor].
          *
          * @see [HttpLoggingInterceptor]
          */
-        @Deprecated(
-            message = "This setting is deprecated. Use customLoggers instead",
-            level = DeprecationLevel.WARNING
-        )
         val httpLoggingInterceptor: HttpLoggingInterceptor?
 
         /**
@@ -500,14 +496,10 @@ interface PNConfiguration : com.pubnub.api.v2.PNConfiguration {
         fun certificatePinner(certificatePinner: CertificatePinner?): Builder
 
         /**
-         * Sets a custom [HttpLoggingInterceptor] for logging network traffic.
+         * Sets a custom [HttpLoggingInterceptor].
          *
          * @see [HttpLoggingInterceptor]
          */
-        @Deprecated(
-            message = "This setting is deprecated. Use customLoggers instead",
-            level = DeprecationLevel.WARNING
-        )
         fun httpLoggingInterceptor(httpLoggingInterceptor: HttpLoggingInterceptor?): Builder
 
         /**

pubnub-gson/pubnub-gson-impl/src/main/java/com/pubnub/internal/java/v2/PNConfigurationImpl.kt

@@ -317,10 +317,6 @@ class PNConfigurationImpl(
 
             override var certificatePinner: CertificatePinner? = defaultConfiguration.certificatePinner
 
-            @Deprecated(
-                message = "This setting is deprecated. Use customLoggers instead.",
-                level = DeprecationLevel.WARNING
-            )
             override fun httpLoggingInterceptor(httpLoggingInterceptor: HttpLoggingInterceptor?): Builder {
                 this.httpLoggingInterceptor = httpLoggingInterceptor
                 return this

pubnub-kotlin/pubnub-kotlin-core-api/src/jvmMain/kotlin/com/pubnub/api/v2/PNConfiguration.kt

@@ -238,14 +238,10 @@ actual interface PNConfiguration {
     val certificatePinner: CertificatePinner?
 
     /**
-     * Sets a custom [HttpLoggingInterceptor] for logging network traffic.
+     * Sets a custom [HttpLoggingInterceptor].
      *
      * @see [HttpLoggingInterceptor]
      */
-    @Deprecated(
-        message = "This setting is deprecated. Use customLoggers instead.",
-        level = DeprecationLevel.WARNING
-    )
     val httpLoggingInterceptor: HttpLoggingInterceptor?
 
     /**
@@ -564,14 +560,10 @@ actual interface PNConfiguration {
         var certificatePinner: CertificatePinner?
 
         /**
-         * Sets a custom [HttpLoggingInterceptor] for logging network traffic.
+         * Sets a custom [HttpLoggingInterceptor].
          *
          * @see [HttpLoggingInterceptor]
          */
-        @Deprecated(
-            message = "This setting is deprecated. Use customLoggers instead",
-            level = DeprecationLevel.WARNING
-        )
         var httpLoggingInterceptor: HttpLoggingInterceptor?
 
         /**

pubnub-kotlin/pubnub-kotlin-impl/src/integrationTest/kotlin/com/pubnub/api/integration/SubscribeIntegrationTests.kt

@@ -1254,7 +1254,7 @@ class SubscribeIntegrationTests : BaseIntegrationTest() {
     }
 
     @Test
-    fun `shouldDeduplicateChannelSubscriptionsWhenSubscribingToSameChannelMultipleTimes`() {
+    fun shouldDeduplicateChannelSubscriptionsWhenSubscribingToSameChannelMultipleTimes() {
         // given
         val numberOfSubscribe = 4
         // punbub.subscribe does subscribe to already subscribed channel so only two subscribe calls should occur. Handshake and actual subscribe.
@@ -1306,7 +1306,7 @@ class SubscribeIntegrationTests : BaseIntegrationTest() {
     }
 
     @Test
-    fun `heartbeatShouldDeduplicateChannelNameInUrlWhenSubscribingToSameChannelMultipleTimes`() {
+    fun heartbeatShouldDeduplicateChannelNameInUrlWhenSubscribingToSameChannelMultipleTimes() {
         // given
         val numberOfSubscribe = 4
         val countDownLatch = CountDownLatch(2) // we want to verify second heartbeat URL
@@ -1347,14 +1347,13 @@ class SubscribeIntegrationTests : BaseIntegrationTest() {
             val channelList = channelsParam.split(",").filter { it.isNotEmpty() }
 
             assertEquals(1, channelList.count { it == testChannel })
-
         } finally {
             pubnub.forceDestroy()
         }
     }
 
     @Test
-    fun `shouldDeduplicateChannelSubscriptionsWhenSubscribingToListOfTheSameChannels`() {
+    fun shouldDeduplicateChannelSubscriptionsWhenSubscribingToListOfTheSameChannels() {
         // given
         val countDownLatch = CountDownLatch(2) // Only two subscribe calls should occur. Handshake and actual subscribe.
         var interceptedUrl: HttpUrl? = null

pubnub-kotlin/pubnub-kotlin-impl/src/main/kotlin/com/pubnub/internal/logging/ConfigurationLogger.kt

@@ -75,7 +75,7 @@ object ConfigurationLogger {
             "proxySelector" to (configuration.proxySelector?.toString() ?: NOT_SET),
             "proxyAuthenticator" to (configuration.proxyAuthenticator?.toString() ?: NOT_SET),
             "maximumConnections" to (configuration.maximumConnections?.toString() ?: NOT_SET),
-            "httpLoggingInterceptor" to (configuration.httpLoggingInterceptor?.let { "(@Deprecated) enabled (${it.level})" } ?: NOT_SET),
+            "httpLoggingInterceptor" to (configuration.httpLoggingInterceptor?.toString() ?: NOT_SET),
             // SSL/TLS settings
             "sslSocketFactory" to (configuration.sslSocketFactory?.toString() ?: NOT_SET),
             "x509ExtendedTrustManager" to (configuration.x509ExtendedTrustManager?.toString() ?: NOT_SET),

pubnub-kotlin/pubnub-kotlin-impl/src/main/kotlin/com/pubnub/internal/v2/PNConfigurationImpl.kt

@@ -189,10 +189,6 @@ class PNConfigurationImpl(
 
         override var certificatePinner: CertificatePinner? = defaultConfiguration.certificatePinner
 
-        @Deprecated(
-            message = "This setting is deprecated. Use customLoggers instead.",
-            level = DeprecationLevel.WARNING
-        )
         override var httpLoggingInterceptor: HttpLoggingInterceptor? = defaultConfiguration.httpLoggingInterceptor
 
         override var sslSocketFactory: SSLSocketFactory? = defaultConfiguration.sslSocketFactory