fix: Bug with ssl network connections + Java module permissions. (#1182)

Closes #1181.
MobilityData · Jun 7, 2022 · e00672c · e00672c
1 parent da57dbf
commit e00672c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 5 deletions.
diff --git a/app/pkg/build.gradle b/app/pkg/build.gradle
@@ -57,6 +57,9 @@ extraJavaModuleInfo {
         requires('java.prefs')
         requires('java.security.jgss')
         requires('java.sql')
+        // Need for SSL support for network connections.
+        requires('jdk.crypto.ec')
+        requires('jdk.crypto.cryptoki')
   }
 }
 

diff --git a/main/src/main/java/org/mobilitydata/gtfsvalidator/util/VersionResolver.java b/main/src/main/java/org/mobilitydata/gtfsvalidator/util/VersionResolver.java
@@ -86,11 +86,16 @@ public synchronized void resolve() {
 
     executor.submit(
         () -> {
-          Optional<String> currentVersion = resolveCurrentVersion();
-          Optional<String> latestReleaseVersion = resolveLatestReleaseVersion();
-          VersionInfo info = VersionInfo.create(currentVersion, latestReleaseVersion);
-          resolvedVersionInfo.set(info);
-          return info;
+          try {
+            Optional<String> currentVersion = resolveCurrentVersion();
+            Optional<String> latestReleaseVersion = resolveLatestReleaseVersion();
+            VersionInfo info = VersionInfo.create(currentVersion, latestReleaseVersion);
+            resolvedVersionInfo.set(info);
+            return info;
+          } catch (Throwable ex) {
+            logger.atSevere().withCause(ex).log("Error resolving version info");
+          }
+          return VersionInfo.empty();
         });
   }