Merge pull request #949 from RADAR-base/bugfix/http-client-url

pvannierop · web-flow · commit aa7c32000fc8 · 2024-09-05T09:07:06.000+02:00
[bugfix] Pass url to HttpClient
diff --git a/radar-auth/src/main/java/org/radarbase/auth/jwks/JwksTokenVerifierLoader.kt b/radar-auth/src/main/java/org/radarbase/auth/jwks/JwksTokenVerifierLoader.kt
@@ -52,7 +52,7 @@ class JwksTokenVerifierLoader(
 
     private suspend fun fetchPublicKeyInfo(): JsonWebKeySet = withContext(Dispatchers.IO) {
         logger.info("Getting the JWT public key at {}", url)
-        val response = httpClient.request()
+        val response = httpClient.request(url)
 
         if (!response.status.isSuccess()) {
             throw TokenValidationException("Cannot fetch token keys (${response.status}) - ${response.bodyAsText()}")
@@ -91,7 +91,6 @@ class JwksTokenVerifierLoader(
                 })
             }
             defaultRequest {
-                url(this@JwksTokenVerifierLoader.url)
                 accept(ContentType.Application.Json)
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ class JwksTokenVerifierLoader(`
`52`	`52`
`53`	`53`	`private suspend fun fetchPublicKeyInfo(): JsonWebKeySet = withContext(Dispatchers.IO) {`
`54`	`54`	`logger.info("Getting the JWT public key at {}", url)`
`55`		`- val response = httpClient.request()`
	`55`	`+ val response = httpClient.request(url)`
`56`	`56`
`57`	`57`	`if (!response.status.isSuccess()) {`
`58`	`58`	`throw TokenValidationException("Cannot fetch token keys (${response.status}) - ${response.bodyAsText()}")`
`@@ -91,7 +91,6 @@ class JwksTokenVerifierLoader(`
`91`	`91`	`})`
`92`	`92`	`}`
`93`	`93`	`defaultRequest {`
`94`		`- url(this@JwksTokenVerifierLoader.url)`
`95`	`94`	`accept(ContentType.Application.Json)`
`96`	`95`	`}`
`97`	`96`	`}`