Merge pull request #88 from Kas-tle/feature/prefer-limbo-yml

Prefer use of limbo.yml for plugin loading

README.md

@@ -1,6 +1,6 @@
 # Limbo
 [![Build Status](http://ci.loohpjames.com/job/Limbo/badge/icon)](http://ci.loohpjames.com/job/Limbo/)
-## Standalone Limbo Minecraft Server (Currently 1.21.7)
+## Standalone Limbo Minecraft Server (Currently 1.21.8)
 
 https://www.spigotmc.org/resources/82468/
 
@@ -18,7 +18,7 @@ IP: mc.loohpjames.com
 ```
 ![Server Banner](https://api.loohpjames.com/serverbanner.png?ip=mc.loohpjames.com&width=918&name=IP:%20mc.loohpjames.com)
 ***
-### Downloads (1.17.1-1.21.7)
+### Downloads (1.17.1-1.21.8)
 - [Jenkins](http://ci.loohpjames.com/job/Limbo/)
 ***
 ### Offical Plugins

pom.xml

@@ -136,7 +136,7 @@
 				</executions>
 			</plugin>
 		</plugins>
-		<finalName>${project.artifactId}-${project.version}-1.21.7</finalName>
+		<finalName>${project.artifactId}-${project.version}-1.21.8</finalName>
 	</build>
 
 	<profiles>
@@ -229,7 +229,7 @@
 		<dependency>
 			<groupId>org.apache.commons</groupId>
 			<artifactId>commons-lang3</artifactId>
-			<version>3.18.0</version>
+			<version>3.14.0</version>
 		</dependency>
 		<dependency>
 			<groupId>com.github.Querz</groupId>

src/main/java/com/loohp/limbo/Limbo.java

@@ -132,7 +132,7 @@ public final class Limbo {
 	
 	//===========================
 	
-	public final String SERVER_IMPLEMENTATION_VERSION = "1.21.7";
+	public final String SERVER_IMPLEMENTATION_VERSION = "1.21.8";
 	public final int SERVER_IMPLEMENTATION_PROTOCOL = 772;
 	public final String LIMBO_IMPLEMENTATION_VERSION;
 	

src/main/java/com/loohp/limbo/plugins/PluginManager.java

@@ -55,18 +55,35 @@ public class PluginManager {
     protected void loadPlugins() {
         for (File file : pluginFolder.listFiles()) {
             if (file.isFile() && file.getName().endsWith(".jar")) {
-				boolean found = false;
                 try (ZipInputStream zip = new ZipInputStream(new FileInputStream(file))) {
+                    ZipEntry limboYmlEntry = null;
+                    ZipEntry pluginYmlEntry = null;
+
                     while (true) {
                         ZipEntry entry = zip.getNextEntry();
                         if (entry == null) {
                             break;
                         }
                         String name = entry.getName();
-						if (name.endsWith("plugin.yml") || name.endsWith("limbo.yml")) {
-							found = true;
+                        if (name.endsWith("limbo.yml")) {
+                            limboYmlEntry = entry;
+                        } else if (name.endsWith("plugin.yml")) {
+                            pluginYmlEntry = entry;
+                        }
+                    }
 
-							FileConfiguration pluginYaml = new FileConfiguration(zip);
+                    ZipEntry chosenEntry = limboYmlEntry != null ? limboYmlEntry : pluginYmlEntry;
+
+                    if (chosenEntry != null) {
+                        try (ZipInputStream processZip = new ZipInputStream(new FileInputStream(file))) {
+                            while (true) {
+                                ZipEntry currentEntry = processZip.getNextEntry();
+                                if (currentEntry == null) {
+                                    break;
+                                }
+
+                                if (currentEntry.getName().equals(chosenEntry.getName())) {
+                                    FileConfiguration pluginYaml = new FileConfiguration(processZip);
                                     String main = pluginYaml.get("main", String.class);
                                     String pluginName = pluginYaml.get("name", String.class);
 
@@ -74,7 +91,8 @@ public class PluginManager {
                                         System.err.println("Ambiguous plugin name in " + file.getName() + " with the plugin \"" + plugins.get(pluginName).getClass().getName() + "\"");
                                         break;
                                     }
-							URLClassLoader child = new URLClassLoader(new URL[] {file.toURI().toURL()}, Limbo.getInstance().getClass().getClassLoader());
+
+                                    URLClassLoader child = new URLClassLoader(new URL[]{file.toURI().toURL()}, Limbo.getInstance().getClass().getClassLoader());
                                     Class<?> clazz = Class.forName(main, true, child);
                                     LimboPlugin plugin = (LimboPlugin) clazz.getDeclaredConstructor().newInstance();
                                     plugin.setInfo(pluginYaml, file);
@@ -84,13 +102,14 @@ public class PluginManager {
                                     break;
                                 }
                             }
+                        }
+                    } else {
+                        System.err.println("Jar file " + file.getName() + " has no plugin.yml or limbo.yml!");
+                    }
                 } catch (Exception e) {
                     System.err.println("Unable to load plugin \"" + file.getName() + "\"");
                     e.printStackTrace();
                 }
-				if (!found) {
-					System.err.println("Jar file " + file.getName() + " has no plugin.yml!");
-				}
             }
         }
     }