Do not provide mod versions which do not match the client's game version

Author: mattco98

build.gradle.kts

@@ -1,7 +1,8 @@
 import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
 
 plugins {
-    kotlin("jvm") version "1.6.0"
+    kotlin("jvm") version "1.8.21"
+    kotlin("plugin.serialization") version "1.8.21"
 }
 
 group = "com.chattriggers"
@@ -13,7 +14,8 @@ repositories {
 }
 
 dependencies {
-    implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.6.0")
+    implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.8.21")
+    implementation("org.jetbrains.kotlinx:kotlinx-serialization-json:1.5.1")
     implementation("club.minnced:discord-webhooks:0.8.0")
     implementation("io.javalin:javalin:4.5.0")
     implementation("com.zaxxer:HikariCP:5.0.1")

src/main/kotlin/com/chattriggers/website/api/versions.kt

@@ -3,6 +3,7 @@ package com.chattriggers.website.api
 import com.fasterxml.jackson.core.Version
 import io.javalin.apibuilder.ApiBuilder.get
 import io.javalin.http.Context
+import kotlinx.serialization.json.Json
 import java.io.File
 
 fun versionRoutes() {
@@ -17,11 +18,21 @@ var lastVersionsCheckTime = System.currentTimeMillis()
 fun getVersions(ctx: Context) {
     if (System.currentTimeMillis() - lastVersionsCheckTime > VERSIONS_CHECK_TIMEOUT) {
         allowedVersions = loadVersions()
-
         lastVersionsCheckTime = System.currentTimeMillis()
     }
 
-    val versions = allowedVersions.groupBy({
+    // Only consider mod versions which are available for the requested version. The
+    // frontend uses the value "all" to signify it wants every version
+    val requestModVersion = ctx.queryParam("modVersion") ?: "1.8.9"
+    val versionsToConsider = if (requestModVersion == "all") {
+        allowedVersions.keys
+    } else {
+        allowedVersions.filter {(_, values) ->
+            values.any { it == requestModVersion }
+        }.keys
+    }
+
+    val versions = versionsToConsider.groupBy({
         "${it.majorVersion}.${it.minorVersion}"
     }, { version ->
         val snapshot = if (version.isSnapshot) {
@@ -33,8 +44,7 @@ fun getVersions(ctx: Context) {
     ctx.status(200).json(versions)
 }
 
-private fun loadVersions() = File(VERSIONS_FILE)
-    .readText()
-    .split("\n")
-    .filter { it.isNotBlank() }
-    .map { it.trim().toVersion() }
+private fun loadVersions(): Map<Version, List<String>> {
+    return Json.decodeFromString<Map<String, List<String>>>(File(VERSIONS_FILE).readText())
+        .mapKeys { it.key.toVersion() }
+}

versions.txt

@@ -1,10 +1,13 @@
-2.0.0
-1.3.0
-1.2.2
-1.2.1
-1.2.0
-1.1.2
-1.1.1
-1.1.0
-1.0.0
-0.18.4
+{
+    "2.0.0": ["1.8.9"],
+    "1.3.0": ["1.8.9"],
+    "1.2.2": ["1.8.9"],
+    "1.2.1": ["1.8.9"],
+    "1.2.0": ["1.8.9"],
+    "1.1.2": ["1.8.9"],
+    "1.1.1": ["1.8.9"],
+    "1.1.0": ["1.8.9"],
+    "1.0.0": ["1.8.9"],
+    "0.18.4": ["1.8.9"],
+    "3.0.0-beta": ["1.19.4"]
+}