fix: when Preemptive Authentication is enabled, basic auth creds were…

… not being set correctly #1764
pact-foundation · Apr 22, 2024 · ac328e2 · ac328e2
1 parent a1d589c
commit ac328e2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/core/pactbroker/src/main/kotlin/au/com/dius/pact/core/pactbroker/HalClient.kt b/core/pactbroker/src/main/kotlin/au/com/dius/pact/core/pactbroker/HalClient.kt
@@ -16,6 +16,7 @@ import au.com.dius.pact.core.support.jsonObject
 import au.com.dius.pact.core.support.unwrap
 import com.google.common.net.UrlEscapers
 import io.github.oshai.kotlinlogging.KLogging
+import org.apache.hc.client5.http.auth.AuthScope
 import org.apache.hc.client5.http.classic.methods.HttpGet
 import org.apache.hc.client5.http.classic.methods.HttpPost
 import org.apache.hc.client5.http.classic.methods.HttpPut
@@ -225,18 +226,19 @@ open class HalClient @JvmOverloads constructor(
         logger.warn { "Authentication options needs to be either an instance of Auth or a list of values, ignoring." }
       }
       val uri = URI(baseUrl)
-      val result = HttpClient.newHttpClient(options["authentication"], uri, this.maxPublishRetries,
+      val (client, credentialsProvider) = HttpClient.newHttpClient(options["authentication"], uri, this.maxPublishRetries,
         this.publishRetryInterval, config.insecureTLS)
-      httpClient = result.first
+      httpClient = client
 
       if (System.getProperty(PREEMPTIVE_AUTHENTICATION) == "true") {
         val targetHost = HttpHost(uri.scheme, uri.host, uri.port)
         logger.warn { "Using preemptive basic authentication with the pact broker at $targetHost" }
         val authCache = BasicAuthCache()
         val basicAuth = BasicScheme()
-        authCache.put(targetHost, basicAuth)
         httpContext = HttpClientContext.create()
-        httpContext!!.credentialsProvider = result.second
+        httpContext!!.credentialsProvider = credentialsProvider
+        basicAuth.initPreemptive(credentialsProvider!!.getCredentials(AuthScope(uri.host, uri.port), httpContext))
+        authCache.put(targetHost, basicAuth)
         httpContext!!.authCache = authCache
       }
     }

diff --git a/core/pactbroker/src/test/groovy/au/com/dius/pact/core/pactbroker/HalClientSpec.groovy b/core/pactbroker/src/test/groovy/au/com/dius/pact/core/pactbroker/HalClientSpec.groovy
@@ -73,11 +73,14 @@ class HalClientSpec extends Specification {
 
     when:
     client.setupHttpClient()
+    def authScheme = client.httpContext.authCache.get(host)
 
     then:
     client.httpClient.credentialsProvider instanceof SystemDefaultCredentialsProvider
     client.httpContext != null
-    client.httpContext.authCache.get(host) instanceof BasicScheme
+    authScheme instanceof BasicScheme
+    authScheme.username == '1'
+    authScheme.password == ['2']
   }
 
   def 'retry strategy is added to execution chain of client'() {