Merge pull request PaperMC#187 from rtm516/fix/buildnumber

Switch to `@Positive` instead of `@Pattern` for buildNumber
GeyserMC · Feb 18, 2024 · 8aacd29 · 8aacd29
2 parents e7181a7 + 4ea51c4
commit 8aacd29
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/main/java/io/papermc/bibliothek/controller/v2/DownloadController.java b/src/main/java/io/papermc/bibliothek/controller/v2/DownloadController.java
@@ -42,6 +42,7 @@
 import io.swagger.v3.oas.annotations.media.Schema;
 import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import jakarta.validation.constraints.Pattern;
+import jakarta.validation.constraints.Positive;
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
@@ -121,7 +122,7 @@ public ResponseEntity<?> download(
     final String versionName,
     @Parameter(description = "A build of the version.")
     @PathVariable("build")
-    @Pattern(regexp = "\\d+") //
+    @Positive //
     final int buildNumber,
     @Parameter(description = "A download of the build.")
     @PathVariable("download")

diff --git a/src/main/java/io/papermc/bibliothek/controller/v2/VersionBuildController.java b/src/main/java/io/papermc/bibliothek/controller/v2/VersionBuildController.java
@@ -39,6 +39,7 @@
 import io.swagger.v3.oas.annotations.media.Schema;
 import io.swagger.v3.oas.annotations.responses.ApiResponse;
 import jakarta.validation.constraints.Pattern;
+import jakarta.validation.constraints.Positive;
 import java.time.Duration;
 import java.time.Instant;
 import java.util.List;
@@ -91,7 +92,7 @@ public ResponseEntity<?> build(
     final String versionName,
     @Parameter(description = "A build of the version.")
     @PathVariable("build")
-    @Pattern(regexp = "\\d+") //
+    @Positive //
     final int buildNumber
   ) {
     final Project project = this.projects.findByName(projectName).orElseThrow(ProjectNotFound::new);