chore: update to 1.20.5

build.gradle.kts

@@ -1,13 +1,38 @@
-import io.gitlab.jfronny.scripts.*
-
 plugins {
-    id("jfmod") version "1.3-SNAPSHOT"
+    id("jfmod") version "1.6-SNAPSHOT"
+}
+
+allprojects { group = "io.gitlab.jfronny" }
+base.archivesName = "dynres"
+
+jfMod {
+    minecraftVersion = "1.20.5"
+    yarn("build.1")
+    loaderVersion = "0.15.10"
+    libJfVersion = "3.15.4"
+    fabricApiVersion = "0.97.6+1.20.5"
+
+    modrinth {
+        projectId = "dynres"
+        requiredDependencies.add("libjf")
+        optionalDependencies.add("modmenu")
+    }
+
+    curseforge {
+        projectId = "407012"
+        requiredDependencies.add("libjf")
+        optionalDependencies.add("modmenu")
+    }
 }
 
 dependencies {
-    modImplementation("io.gitlab.jfronny.libjf:libjf-web-v0:${prop("libjf_version")}")
-    modImplementation("io.gitlab.jfronny.libjf:libjf-config-core-v1:${prop("libjf_version")}")
+    modImplementation("io.gitlab.jfronny.libjf:libjf-web-v1")
+    modImplementation("io.gitlab.jfronny.libjf:libjf-config-core-v2")
 
     // Dev env
-    modLocalRuntime("com.terraformersmc:modmenu:6.1.0-rc.1")
+    modLocalRuntime("com.terraformersmc:modmenu:10.0.0-beta.1")
+    // for modmenu
+    modLocalRuntime("net.fabricmc.fabric-api:fabric-resource-loader-v0")
+    modLocalRuntime("net.fabricmc.fabric-api:fabric-screen-api-v1")
+    modLocalRuntime("net.fabricmc.fabric-api:fabric-key-binding-api-v1")
 }

gradle.properties

@@ -1,17 +0,0 @@
-# check these on https://fabricmc.net/develop/
-minecraft_version=1.19.4
-yarn_mappings=build.1
-loader_version=0.14.17
-
-maven_group=io.gitlab.jfronny
-archives_base_name=dynres
-
-modrinth_id=dynres
-modrinth_required_dependencies=libjf
-modrinth_optional_dependencies=modmenu
-curseforge_id=407012
-curseforge_required_dependencies=libjf
-curseforge_optional_dependencies=modmenu
-
-fabric_version=0.75.3+1.19.4
-libjf_version=3.4.1

src/client/java/io/gitlab/jfronny/dynres/mixin/ClientCommonNetworkHandlerMixin.java

@@ -0,0 +1,33 @@
+package io.gitlab.jfronny.dynres.mixin;
+
+import io.gitlab.jfronny.dynres.DynRes;
+import net.minecraft.client.network.ClientCommonNetworkHandler;
+import net.minecraft.network.packet.c2s.common.ResourcePackStatusC2SPacket;
+import net.minecraft.network.packet.s2c.common.ResourcePackSendS2CPacket;
+import org.jetbrains.annotations.Nullable;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.injection.*;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
+
+import java.net.URL;
+
+@Mixin(ClientCommonNetworkHandler.class)
+public abstract class ClientCommonNetworkHandlerMixin {
+    @Shadow @Nullable protected static URL getParsedResourcePackUrl(String url) {
+        throw new RuntimeException("Mixin not applied");
+    }
+
+    @Inject(at = @At("HEAD"), method = "onResourcePackSend(Lnet/minecraft/network/packet/s2c/common/ResourcePackSendS2CPacket;)V")
+    public void logResourcePackStatus(ResourcePackSendS2CPacket status, CallbackInfo info) {
+        DynRes.LOGGER.info("[RECEIVE] ResourcePackUrl={0}", status.url());
+        DynRes.LOGGER.info("[RECEIVE] ResourcePackHash={0}", status.hash());
+    }
+
+    @Redirect(method = "onResourcePackSend(Lnet/minecraft/network/packet/s2c/common/ResourcePackSendS2CPacket;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/network/ClientCommonNetworkHandler;getParsedResourcePackUrl(Ljava/lang/String;)Ljava/net/URL;"))
+    public URL logResourcePackStatus(String url) {
+        URL res = getParsedResourcePackUrl(url);
+        DynRes.LOGGER.info("[CHECK] ResourcePackUrlValid={0}", res != null);
+        return res;
+    }
+}

src/client/java/io/gitlab/jfronny/dynres/mixin/ClientPlayNetworkHandlerMixin.java

@@ -1,38 +0,0 @@
-package io.gitlab.jfronny.dynres.mixin;
-
-import io.gitlab.jfronny.dynres.*;
-import net.minecraft.client.network.*;
-import net.minecraft.network.packet.c2s.play.*;
-import net.minecraft.network.packet.s2c.play.*;
-import org.jetbrains.annotations.*;
-import org.spongepowered.asm.mixin.*;
-import org.spongepowered.asm.mixin.injection.*;
-import org.spongepowered.asm.mixin.injection.callback.*;
-
-import java.net.*;
-
-@Mixin(ClientPlayNetworkHandler.class)
-public abstract class ClientPlayNetworkHandlerMixin {
-    @Shadow @Nullable
-    private static URL resolveUrl(String url) {
-        throw new RuntimeException("Mixin not applied");
-    }
-
-    @Inject(at = @At("HEAD"), method = "sendResourcePackStatus(Lnet/minecraft/network/packet/c2s/play/ResourcePackStatusC2SPacket$Status;)V")
-    public void logSentResourcePackStatus(ResourcePackStatusC2SPacket.Status packStatus, CallbackInfo info) {
-        DynRes.LOGGER.info("[SEND] ResourcePackStatus=" + packStatus);
-    }
-
-    @Inject(at = @At("HEAD"), method = "onResourcePackSend(Lnet/minecraft/network/packet/s2c/play/ResourcePackSendS2CPacket;)V")
-    public void logResourcePackStatus(ResourcePackSendS2CPacket status, CallbackInfo info) {
-        DynRes.LOGGER.info("[RECEIVE] ResourcePackUrl=" + status.getURL());
-        DynRes.LOGGER.info("[RECEIVE] ResourcePackSha1=" + status.getSHA1());
-    }
-
-    @Redirect(method = "onResourcePackSend(Lnet/minecraft/network/packet/s2c/play/ResourcePackSendS2CPacket;)V", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/network/ClientPlayNetworkHandler;resolveUrl(Ljava/lang/String;)Ljava/net/URL;"))
-    public URL logResourcePackStatus(String url) {
-        URL res = resolveUrl(url);
-        DynRes.LOGGER.info("[CHECK] ResourcePackUrlValid=" + (res != null));
-        return res;
-    }
-}

src/client/java/io/gitlab/jfronny/dynres/mixin/ServerInfoMixin.java

@@ -11,6 +11,6 @@ import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
 public class ServerInfoMixin {
     @Inject(at = @At("RETURN"), method = "getResourcePackPolicy()Lnet/minecraft/client/network/ServerInfo$ResourcePackPolicy;")
     public void getResPackState(CallbackInfoReturnable<ServerInfo.ResourcePackPolicy> info) {
-        DynRes.LOGGER.info("[RECEIVE] ResourcePackStatus=" + info.getReturnValue());
+        DynRes.LOGGER.info("[RECEIVE] ResourcePackStatus={0}", info.getReturnValue());
     }
 }

src/client/resources/dynres.client.mixins.json

@@ -4,7 +4,7 @@
   "package": "io.gitlab.jfronny.dynres.mixin",
   "compatibilityLevel": "JAVA_17",
   "client": [
-    "ClientPlayNetworkHandlerMixin",
+    "ClientCommonNetworkHandlerMixin",
     "ServerInfoMixin"
   ],
   "injectors": {

src/main/java/io/gitlab/jfronny/dynres/Cfg.java

@@ -1,7 +1,7 @@
 package io.gitlab.jfronny.dynres;
 
-import io.gitlab.jfronny.libjf.config.api.v1.Entry;
-import io.gitlab.jfronny.libjf.config.api.v1.JfConfig;
+import io.gitlab.jfronny.libjf.config.api.v2.Entry;
+import io.gitlab.jfronny.libjf.config.api.v2.JfConfig;
 
 @JfConfig(referencedConfigs = "libjf-web-v0")
 public class Cfg  {

src/main/java/io/gitlab/jfronny/dynres/DynRes.java

@@ -1,18 +1,16 @@
 package io.gitlab.jfronny.dynres;
 
-import io.gitlab.jfronny.libjf.web.api.WebInit;
-import io.gitlab.jfronny.libjf.web.api.WebServer;
+import io.gitlab.jfronny.commons.logger.SystemLoggerPlus;
+import io.gitlab.jfronny.libjf.web.api.v1.*;
 import net.fabricmc.api.EnvType;
 import net.fabricmc.loader.api.FabricLoader;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
 import java.nio.file.Files;
 import java.nio.file.Path;
 
-public class DynRes implements WebInit {
-    public static final Logger LOGGER = LoggerFactory.getLogger("DynRes");
+public class DynRes implements WebEntrypoint {
+    public static final SystemLoggerPlus LOGGER = SystemLoggerPlus.forName("DynRes");
     public static Path packFile;
     public static String packUrl = "";
 
@@ -30,10 +28,10 @@ public class DynRes implements WebInit {
     public void register(WebServer api) {
         if (FabricLoader.getInstance().getEnvironmentType() == EnvType.SERVER && packFile != null) {
             try {
-                packUrl = api.registerFile("/resources.zip", packFile, !Cfg.hashResources);
-                LOGGER.info("Initialized DynRes at " + packUrl);
+                packUrl = api.registerFile(PathSegment.of("resources.zip"), packFile, !Cfg.hashResources);
+                LOGGER.info("Initialized DynRes at {0}", packUrl);
             } catch (IOException e) {
-                e.printStackTrace();
+                LOGGER.error("Could not register DynRes resource pack", e);
             }
         }
     }

src/main/java/io/gitlab/jfronny/dynres/mixin/ServerPropertiesHandlerMixin.java

@@ -1,20 +1,26 @@
 package io.gitlab.jfronny.dynres.mixin;
 
-import io.gitlab.jfronny.dynres.*;
-import io.gitlab.jfronny.libjf.web.api.*;
-import io.gitlab.jfronny.libjf.web.impl.util.*;
-import net.minecraft.server.*;
-import net.minecraft.server.dedicated.*;
-import net.minecraft.text.*;
-import org.jetbrains.annotations.*;
-import org.spongepowered.asm.mixin.*;
-import org.spongepowered.asm.mixin.injection.*;
-import org.spongepowered.asm.mixin.injection.callback.*;
+import io.gitlab.jfronny.dynres.Cfg;
+import io.gitlab.jfronny.dynres.DynRes;
+import io.gitlab.jfronny.libjf.web.api.v1.WebServer;
+import io.gitlab.jfronny.libjf.web.impl.util.WebPaths;
+import net.minecraft.server.MinecraftServer;
+import net.minecraft.server.dedicated.ServerPropertiesHandler;
+import net.minecraft.text.Text;
+import org.jetbrains.annotations.Nullable;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
 
-import java.io.*;
-import java.nio.file.*;
-import java.security.*;
-import java.util.*;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.nio.file.Files;
+import java.security.MessageDigest;
+import java.security.NoSuchAlgorithmException;
+import java.util.Optional;
+import java.util.UUID;
 
 @Mixin(ServerPropertiesHandler.class)
 public abstract class ServerPropertiesHandlerMixin {
@@ -23,8 +29,8 @@ public abstract class ServerPropertiesHandlerMixin {
         throw new IllegalStateException("Mixin not applied");
     }
 
-    @Inject(method = "getServerResourcePackProperties(Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;ZLjava/lang/String;)Ljava/util/Optional;", at = @At("HEAD"), cancellable = true)
-    private static void getServerResourcePackProperties(String url, String sha1, @Nullable String hash, boolean required, String prompt, CallbackInfoReturnable<Optional<MinecraftServer.ServerResourcePackProperties>> cir) {
+    @Inject(method = "getServerResourcePackProperties(Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;ZLjava/lang/String;)Ljava/util/Optional;", at = @At("HEAD"), cancellable = true)
+    private static void getServerResourcePackProperties(String id, String url, String sha1, String hash, boolean required, String prompt, CallbackInfoReturnable<Optional<MinecraftServer.ServerResourcePackProperties>> cir) {
         if (DynRes.packFile != null) {
             sha1 = "";
             if (Cfg.hashResources) {
@@ -34,13 +40,15 @@ public abstract class ServerPropertiesHandlerMixin {
                         result.append(Integer.toString((b & 0xff) + 0x100, 16).substring(1));
                     }
                     sha1 = result.toString();
-                    DynRes.LOGGER.info("Set hash to " + sha1);
+                    DynRes.LOGGER.info("Set hash to {0}", sha1);
                 } catch (IOException | NoSuchAlgorithmException e) {
                     DynRes.LOGGER.error("Failed to get hash, continuing with empty", e);
                 }
             }
+            url = WebPaths.concat(WebServer.getInstance().getServerRoot(), "resources.zip");
             cir.setReturnValue(Optional.of(new MinecraftServer.ServerResourcePackProperties(
-                    WebPaths.concat(WebServer.getInstance().getServerRoot(), "resources.zip"),
+                    UUID.nameUUIDFromBytes(url.getBytes(StandardCharsets.UTF_8)),
+                    url,
                     sha1,
                     required,
                     parseResourcePackPrompt(prompt)