Merge pull request #24 from digipost/catch-and-log-apollo-exception

Catch ApolloHttpException when fetching vulnerabilities
digipost · Sep 24, 2024 · cba4ea2 · cba4ea2
2 parents 2c333e6 + a5efa30
commit cba4ea2
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/src/main/kotlin/no/digipost/github/monitoring/GithubGraphql.kt b/src/main/kotlin/no/digipost/github/monitoring/GithubGraphql.kt
@@ -4,6 +4,7 @@ import com.apollographql.apollo3.ApolloCall
 import com.apollographql.apollo3.ApolloClient
 import com.apollographql.apollo3.api.ApolloResponse
 import com.apollographql.apollo3.api.Optional
+import com.apollographql.apollo3.exception.ApolloHttpException
 import com.github.graphql.client.GetVulnerabilityAlertsForRepoQuery
 import com.github.graphql.client.QueryRepositoriesQuery
 import kotlinx.coroutines.channels.Channel
@@ -47,12 +48,16 @@ fun fetchAllReposWithVulnerabilities(apolloClient: ApolloClient, githubApiClient
                 repositories.add(r)
 
                 launch {
-                    val vulnerabilities = getVulnerabilitiesForRepo(apolloClient, r.name)
-                    if (vulnerabilities.isNotEmpty()) {
-                        r.copy(vulnerabilities = vulnerabilities).let {
-                            logger.info("{} sårbarheter i {}", vulnerabilities.size, r.name)
-                            vulnRepositories[it.name] = vulnerabilities
+                    try {
+                        val vulnerabilities = getVulnerabilitiesForRepo(apolloClient, r.name)
+                        if (vulnerabilities.isNotEmpty()) {
+                            r.copy(vulnerabilities = vulnerabilities).let {
+                                logger.info("{} sårbarheter i {}", vulnerabilities.size, r.name)
+                                vulnRepositories[it.name] = vulnerabilities
+                            }
                         }
+                    } catch (e: ApolloHttpException) {
+                        logger.warn("ApolloHttpException ved henting av sårbarheter for repo {}", r.name, e)
                     }
                 }