chore: instantiate HttpClient for calling GitHub in server

Author: krzema12

jit-binding-server/src/main/kotlin/io/github/typesafegithub/workflows/jitbindingserver/Main.kt

@@ -10,6 +10,8 @@ import io.github.typesafegithub.workflows.mavenbinding.VersionArtifacts
 import io.github.typesafegithub.workflows.mavenbinding.buildPackageArtifacts
 import io.github.typesafegithub.workflows.mavenbinding.buildVersionArtifacts
 import io.github.typesafegithub.workflows.shared.internal.getGithubAuthToken
+import io.ktor.client.HttpClient
+import io.ktor.client.engine.cio.CIO
 import io.ktor.http.HttpStatusCode
 import io.ktor.server.application.Application
 import io.ktor.server.application.ApplicationCall
@@ -60,11 +62,12 @@ fun main() {
 }
 
 fun Application.appModule(
-    buildVersionArtifacts: suspend (ActionCoords) -> VersionArtifacts?,
+    buildVersionArtifacts: suspend (ActionCoords, HttpClient) -> VersionArtifacts?,
     buildPackageArtifacts: suspend (ActionCoords, String, (Collection<ActionCoords>) -> Unit) -> Map<String, String>,
     getGithubAuthToken: () -> String,
 ) {
-    val bindingsCache = buildBindingsCache(buildVersionArtifacts)
+    val httpClient = HttpClient(CIO)
+    val bindingsCache = buildBindingsCache(buildVersionArtifacts, httpClient)
     val metadataCache = buildMetadataCache(bindingsCache, buildPackageArtifacts, getGithubAuthToken)
     installPlugins(prometheusRegistry)
 
@@ -77,13 +80,14 @@ fun Application.appModule(
 }
 
 private fun buildBindingsCache(
-    buildVersionArtifacts: suspend (ActionCoords) -> VersionArtifacts?,
+    buildVersionArtifacts: suspend (ActionCoords, HttpClient) -> VersionArtifacts?,
+    httpClient: HttpClient,
 ): LoadingCache<ActionCoords, CachedVersionArtifact> =
     Caffeine
         .newBuilder()
         .refreshAfterWrite(1.hours)
         .recordStats()
-        .asLoadingCache<ActionCoords, CachedVersionArtifact> { buildVersionArtifacts(it) }
+        .asLoadingCache<ActionCoords, CachedVersionArtifact> { buildVersionArtifacts(it, httpClient) }
 
 @Suppress("ktlint:standard:function-signature") // Conflict with detekt.
 private fun buildMetadataCache(

maven-binding-builder/build.gradle.kts

@@ -5,6 +5,7 @@ plugins {
 dependencies {
     implementation("org.jetbrains.kotlin:kotlin-compiler")
     api("io.arrow-kt:arrow-core:2.2.0")
+    api("io.ktor:ktor-client-core:3.3.3")
     api(projects.actionBindingGenerator)
     implementation(projects.sharedInternal)
     implementation("io.github.oshai:kotlin-logging:7.0.13")

maven-binding-builder/src/main/kotlin/io/github/typesafegithub/workflows/mavenbinding/JarBuilding.kt

@@ -7,6 +7,7 @@ import io.github.typesafegithub.workflows.actionbindinggenerator.domain.NewestFo
 import io.github.typesafegithub.workflows.actionbindinggenerator.domain.TypingActualSource
 import io.github.typesafegithub.workflows.actionbindinggenerator.generation.ActionBinding
 import io.github.typesafegithub.workflows.actionbindinggenerator.generation.generateBinding
+import io.ktor.client.HttpClient
 import org.jetbrains.kotlin.cli.common.ExitCode
 import org.jetbrains.kotlin.cli.common.arguments.K2JVMCompilerArguments
 import org.jetbrains.kotlin.cli.common.messages.MessageRenderer
@@ -31,9 +32,9 @@ internal data class Jars(
     val typingActualSource: TypingActualSource?,
 )
 
-internal suspend fun ActionCoords.buildJars(): Jars? {
+internal suspend fun ActionCoords.buildJars(httpClient: HttpClient): Jars? {
     val binding =
-        generateBinding(metadataRevision = NewestForVersion).also {
+        generateBinding(metadataRevision = NewestForVersion, httpClient = httpClient).also {
             if (it.isEmpty()) return null
         }
 

maven-binding-builder/src/main/kotlin/io/github/typesafegithub/workflows/mavenbinding/VersionArtifactsBuilding.kt

@@ -2,6 +2,7 @@ package io.github.typesafegithub.workflows.mavenbinding
 
 import io.github.typesafegithub.workflows.actionbindinggenerator.domain.ActionCoords
 import io.github.typesafegithub.workflows.actionbindinggenerator.domain.TypingActualSource
+import io.ktor.client.HttpClient
 
 sealed interface Artifact
 
@@ -18,9 +19,9 @@ data class VersionArtifacts(
     val typingActualSource: TypingActualSource?,
 )
 
-suspend fun buildVersionArtifacts(actionCoords: ActionCoords): VersionArtifacts? {
+suspend fun buildVersionArtifacts(actionCoords: ActionCoords, httpClient: HttpClient): VersionArtifacts? {
     with(actionCoords) {
-        val jars = buildJars() ?: return null
+        val jars = buildJars(httpClient = httpClient) ?: return null
         val pom = buildPomFile()
         val mainJarSize by lazy { jars.mainJar().size }
         val mainJarMd5Checksum by lazy { jars.mainJar().md5Checksum() }