Fix Java 17 compatibility issues - Update to Java 17 and fix getFirst/getLast usage

Co-authored-by: commjoen <[email protected]>

Author: Copilot

pom.xml

@@ -57,13 +57,13 @@
     <gcp.sdk.version>7.2.0</gcp.sdk.version>
     <github.button.version>2.14.1</github.button.version>
     <io.netty.version>4.1.118.Final</io.netty.version>
-    <java.version>23</java.version>
+    <java.version>17</java.version>
     <jquery.version>3.7.1</jquery.version>
     <jruby.version>10.0.2.0</jruby.version>
     <lombok.version>1.18.38</lombok.version>
     <maven-compiler-plugin.version>3.14.0</maven-compiler-plugin.version>
     <maven.compiler.proc>full</maven.compiler.proc>
-    <maven.compiler.target>23</maven.compiler.target>
+    <maven.compiler.target>17</maven.compiler.target>
     <spring.cloud-version>2025.0.0</spring.cloud-version>
     <spring.security.version>6.2.3</spring.security.version>
     <system-stubs-jupiter.version>2.1.8</system-stubs-jupiter.version>
@@ -539,8 +539,8 @@
         <artifactId>maven-compiler-plugin</artifactId>
         <version>${maven-compiler-plugin.version}</version>
         <configuration>
-          <source>23</source>
-          <target>23</target>
+          <source>17</source>
+          <target>17</target>
         </configuration>
       </plugin>
       <plugin>

src/main/java/org/owasp/wrongsecrets/Challenges.java

@@ -100,7 +100,8 @@ public boolean isFirstChallenge(ChallengeDefinition challengeDefinition) {
   }
 
   public boolean isLastChallenge(ChallengeDefinition challengeDefinition) {
-    return challengeDefinition.equals(definitions.challenges().getLast());
+    var challenges = definitions.challenges();
+    return challengeDefinition.equals(challenges.get(challenges.size() - 1));
   }
 
   public List<ChallengeDefinition> getChallengeDefinitions() {

src/main/java/org/owasp/wrongsecrets/challenges/ChallengeUI.java

@@ -109,7 +109,7 @@ private String documentation(Function<ChallengeSource, String> extractor) {
       return challengeDefinition.source(runtimeEnvironment).map(extractor).orElse("");
     } else {
       // We cannot run the challenge but showing documentation should still be possible
-      return extractor.apply(challengeDefinition.sources().getFirst());
+      return extractor.apply(challengeDefinition.sources().get(0));
     }
   }
 

src/main/java/org/owasp/wrongsecrets/challenges/ChallengesController.java

@@ -102,7 +102,7 @@ public String spoiler(@PathVariable("short-name") String shortName, Model model)
       Supplier<Spoiler> spoilerFromRandomChallenge =
           () -> {
             var challengeDefinition = findByShortName(shortName);
-            return challenges.getChallenge(challengeDefinition).getFirst().spoiler();
+            return challenges.getChallenge(challengeDefinition).get(0).spoiler();
           };
 
       // We always want to show the spoiler even if we run in a non-supported environment

src/test/java/org/owasp/wrongsecrets/ctftests/ChallengesControllerCTFClientModeTest.java

@@ -36,23 +36,23 @@ class ChallengesControllerCTFClientModeTest {
 
   @Test
   void shouldNotSpoilWhenInCTFMode() throws Exception {
-    var randomChallenge = challenges.getChallengeDefinitions().getFirst();
+    var randomChallenge = challenges.getChallengeDefinitions().get(0);
     mvc.perform(get("/spoil/%s".formatted(randomChallenge.name().shortName())))
         .andExpect(status().isOk())
         .andExpect(content().string(containsString("Spoils are disabled in CTF mode")));
   }
 
   @Test
   void shouldNotSpoilWhenInCTFModeEvenWhenChallengeUnsupported() throws Exception {
-    var firstChallenge = challenges.getChallengeDefinitions().getFirst();
+    var firstChallenge = challenges.getChallengeDefinitions().get(0);
     mvc.perform(get("/spoil/%s".formatted(firstChallenge.name().shortName())))
         .andExpect(status().isOk())
         .andExpect(content().string(containsString("Spoils are disabled in CTF mode")));
   }
 
   @Test
   void challenge0SshouldSShowTheAddressRightAnswersNeedToBeSubmittedTo() throws Exception {
-    var firstChallenge = challenges.getChallengeDefinitions().getFirst();
+    var firstChallenge = challenges.getChallengeDefinitions().get(0);
     mvc.perform(get("/challenge/%s".formatted(firstChallenge.name().shortName())))
         .andExpect(status().isOk())
         .andExpect(content().string(containsString("https://www.google.nl")));

src/test/java/org/owasp/wrongsecrets/ctftests/ChallengesControllerCTFModeWithPresetCloudValuesTest.java

@@ -42,7 +42,7 @@ class ChallengesControllerCTFModeWithPresetCloudValuesTest {
 
   @Test
   void shouldNotSpoilWhenInCTFMode() throws Exception {
-    var firstChallenge = challenges.getChallengeDefinitions().getFirst();
+    var firstChallenge = challenges.getChallengeDefinitions().get(0);
     mvc.perform(get("/spoil/%s".formatted(firstChallenge.name().shortName())))
         .andExpect(status().isOk())
         .andExpect(content().string(containsString("Spoils are disabled in CTF mode")));
@@ -51,7 +51,7 @@ void shouldNotSpoilWhenInCTFMode() throws Exception {
   @Test
   void shouldShowFlagWhenRespondingWithSuccessInCTFModeChallenge9() throws Exception {
     var challenge9Definition = challenges.findByShortName("challenge-9").orElseThrow();
-    var challenge9 = challenges.getChallenge(challenge9Definition).getFirst();
+    var challenge9 = challenges.getChallenge(challenge9Definition).get(0);
     var spoil = challenge9.spoiler().solution();
     mvc.perform(
             post("/challenge/%s".formatted(challenge9Definition.name().shortName()))

src/test/java/org/owasp/wrongsecrets/definitions/NavigationTest.java

@@ -16,7 +16,7 @@ void navigatePreviousWhenOnFirstChallenge() {
     var navigation =
         new Navigator(
             challengeDefinitionsConfiguration.challenges(),
-            challengeDefinitionsConfiguration.challenges().getFirst());
+            challengeDefinitionsConfiguration.challenges().get(0));
 
     assertThat(navigation.previous()).isEmpty();
   }
@@ -26,14 +26,14 @@ void navigateNextWhenOnLastChallenge() {
     var navigation =
         new Navigator(
             challengeDefinitionsConfiguration.challenges(),
-            challengeDefinitionsConfiguration.challenges().getLast());
+            challengeDefinitionsConfiguration.challenges().get(challengeDefinitionsConfiguration.challenges().size() - 1));
 
     assertThat(navigation.next()).isEmpty();
   }
 
   @Test
   void navigatePreviousAndNextOnSecondChallenge() {
-    var first = challengeDefinitionsConfiguration.challenges().getFirst();
+    var first = challengeDefinitionsConfiguration.challenges().get(0);
     var second = challengeDefinitionsConfiguration.challenges().get(1);
     var third = challengeDefinitionsConfiguration.challenges().get(2);