litminions

[Yamiru]

console error:
[14:45:17 ERROR]: Could not pass event IslandDeleteEvent to LitLibs v1.2.2
java.lang.IllegalArgumentException: UUID string too large
at java.base/java.util.UUID.fromString1(UUID.java:266) ~[?:?]
at java.base/java.util.UUID.fromString(UUID.java:260) ~[?:?]
at LitLibs-1.2.2.jar/me.waterarchery.litlibs.hooks.protection.BentoBoxHook.parseUUID(BentoBoxHook.java:128) ~[LitLibs-1.2.2.jar:?]
at LitLibs-1.2.2.jar/me.waterarchery.litlibs.hooks.protection.BentoBoxHook.onIslandDisband(BentoBoxHook.java:113) ~[LitLibs-1.2.2.jar:?]
at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:80) ~[paper-api-1.21.10-R0.1-SNAPSHOT.jar:?]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:71) ~[paper-api-1.21.10-R0.1-SNAPSHOT.jar:?]
at io.papermc.paper.plugin.manager.PaperEventManager.callEvent(PaperEventManager.java:54) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at io.papermc.paper.plugin.manager.PaperPluginManagerImpl.callEvent(PaperPluginManagerImpl.java:131) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:628) ~[paper-api-1.21.10-R0.1-SNAPSHOT.jar:?]
at BentoBox-3.10.2-SNAPSHOT-b2997.jar/world.bentobox.bentobox.api.events.island.IslandEvent$IslandEventBuilder.build(IslandEvent.java:387) ~[BentoBox-3.10.2-SNAPSHOT-b2997.jar:?]
at BentoBox-3.10.2-SNAPSHOT-b2997.jar/world.bentobox.bentobox.managers.IslandsManager.deleteIsland(IslandsManager.java:293) ~[BentoBox-3.10.2-SNAPSHOT-b2997.jar:?]
at BentoBox-3.10.2-SNAPSHOT-b2997.jar/world.bentobox.bentobox.managers.island.NewIsland.tidyUp(NewIsland.java:374) ~[BentoBox-3.10.2-SNAPSHOT-b2997.jar:?]
at BentoBox-3.10.2-SNAPSHOT-b2997.jar/world.bentobox.bentobox.managers.island.NewIsland.lambda$postCreationTask$3(NewIsland.java:286) ~[BentoBox-3.10.2-SNAPSHOT-b2997.jar:?]
at java.base/java.util.concurrent.CompletableFuture$UniRun.tryFire(CompletableFuture.java:787) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture.complete(CompletableFuture.java:2179) ~[?:?]
at BentoBox-3.10.2-SNAPSHOT-b2997.jar/world.bentobox.bentobox.managers.IslandsManager.lambda$homeTeleportAsync$19(IslandsManager.java:1120) ~[BentoBox-3.10.2-SNAPSHOT-b2997.jar:?]
at java.base/java.util.concurrent.CompletableFuture$UniAccept.tryFire(CompletableFuture.java:718) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510) ~[?:?]
at java.base/java.util.concurrent.CompletableFuture.complete(CompletableFuture.java:2179) ~[?:?]
at org.bukkit.craftbukkit.entity.CraftEntity.lambda$teleportAsync$4(CraftEntity.java:1132) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.TickTask.run(TickTask.java:18) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.util.thread.BlockableEventLoop.doRunTask(BlockableEventLoop.java:176) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(ReentrantBlockableEventLoop.java:24) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:1537) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.doRunTask(MinecraftServer.java:180) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.util.thread.BlockableEventLoop.pollTask(BlockableEventLoop.java:150) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.pollTaskInternal(MinecraftServer.java:1517) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.pollTask(MinecraftServer.java:1511) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.recordTaskExecutionTimeWhileWaiting(MinecraftServer.java:1229) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.runServer(MinecraftServer.java:1352) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at net.minecraft.server.MinecraftServer.lambda$spin$2(MinecraftServer.java:384) ~[paper-1.21.10.jar:1.21.10-117-df4b668]
at java.base/java.lang.Thread.run(Thread.java:1583) ~[?:?]

[sys-arch]

Clarifying the issue submitted by @Yamiru :

This error occurs when purging player islands using BentoBox's aOneBlock plugin, which reports that the UUIDs are too long. The plugin adds a prefix to the island UUIDs, presumably to differentiate them from player UUIDs.

This occurred on a Paper 1.21.10 server on Linux with the Litminions plugin and the one mentioned above.

[sys-arch]

@WaterArchery WIll you solve this?

[Nrleryx]

"Hi, I've investigated the 'UUID string too large' error occurring during the IslandDeleteEvent.

The issue is in the BentoBoxHook.java file, specifically within the parseUUID method (line 128). When using BentoBox with certain addons like aOneBlock, the island IDs are not always standard 36-character UUIDs; they often include prefixes or unique identifiers that make the string longer. Since the code tries to use UUID.fromString() directly on this raw string, it throws an IllegalArgumentException.

To fix this, the parseUUID method should probably be updated to handle non-standard ID formats or use BentoBox's internal methods to retrieve the clean UUID.

Could you please check this hook implementation? Thanks."

[sys-arch]

The issue is in the BentoBoxHook.java file, specifically within the parseUUID method (line 128). When using BentoBox with certain addons like aOneBlock, the island IDs are not always standard 36-character UUIDs; they often include prefixes or unique identifiers that make the string longer. Since the code tries to use UUID.fromString() directly on this raw string, it throws an IllegalArgumentException.

To fix this, the parseUUID method should probably be updated to handle non-standard ID formats or use BentoBox's internal methods to retrieve the clean UUID.

Thank you, @Nrleryx for your contribution but i think the solution is more simple than what you said. Since this file only process Bentobox plugins (SkyBlok and OneBlock) only is nedeed change this line:

String cleanedUUID = rawUUID.replace("BSkyBlock", "");

for this one:

String cleanedUUID = rawUUID.replaceAll("BSkyBlock|AOneBlock", "");

But imagine that tomorrow BentoBox launch new plugins so the last solution is not valid so we can use patterns like this one;

private static final Pattern UUID_PATTERN = 
    Pattern.compile("([a-fA-F0-9]{8}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{4}-[a-fA-F0-9]{12})");

private UUID parseUUID(String rawUUID) {
    Matcher matcher = UUID_PATTERN.matcher(rawUUID);
    if (matcher.find()) {
        return UUID.fromString(matcher.group(1));
    }
    throw new IllegalArgumentException("Not valid UUID found on: " + rawUUID);
}

Is a good solution ensuring secuirty but if you want a better performance solution, we can use substring:

private UUID parseUUID(String rawUUID) {
    if (rawUUID.length() < 36) {
        throw new IllegalArgumentException("Very short string to contains an UUID");
    }
    // Cortamos los últimos 36 caracteres
    String cleanedUUID = rawUUID.substring(rawUUID.length() - 36);
    return UUID.fromString(cleanedUUID);
}

[Nrleryx]

I agree with @sys-arch the regex pattern or the substring method is a much more robust solution than hardcoding specific plugin names. This way, the hook will remain compatible even if BentoBox adds new game modes in the future.

@WaterArchery , could you please consider implementing one of these suggested fixes in the next update? The substring approach seems more efficient, while regex is safer for validation.