1.1.1

* actually fix logging

Author: phit

build.gradle

@@ -8,7 +8,7 @@ plugins {
 }
 
 group = 'de.sldk.mc.prometheusexporter'
-version = '1.1.0'
+version = '1.1.1'
 description = "Sponge Minecraft Prometheus Exporter"
 ext.url = 'https://github.com/Stonebound/minecraft-prometheus-exporter'
 

src/main/java/de/sldk/mc/prometheusexporter/JettyNullLogger.java

@@ -3,52 +3,19 @@
 import org.eclipse.jetty.util.log.Logger;
 
 public class JettyNullLogger implements Logger {
-    public String getName() {
-        return "Jetty";
-    }
-
-    public void warn(String s, Object... objects) {
-    }
-
-    public void warn(Throwable throwable) {
-    }
-
-    public void warn(String s, Throwable throwable) {
-    }
-
-    public void info(String s, Object... objects) {
-    }
-
-    public void info(Throwable throwable) {
-    }
-
-    public void info(String s, Throwable throwable) {
-    }
-
-    public boolean isDebugEnabled() {
-        return false;
-    }
-
-    public void setDebugEnabled(boolean b) {
-    }
-
-    public void debug(String s, Object... objects) {
-    }
-
-    public void debug(String msg, long value) {
-
-    }
-
-    public void debug(Throwable throwable) {
-    }
-
-    public void debug(String s, Throwable throwable) {
-    }
-
-    public Logger getLogger(String s) {
-        return this;
-    }
-
-    public void ignore(Throwable throwable) {
-    }
+    @Override public String getName() { return "Jetty"; }
+    @Override public void warn(String msg, Object... args) { }
+    @Override public void warn(Throwable thrown) { }
+    @Override public void warn(String msg, Throwable thrown) { }
+    @Override public void info(String msg, Object... args) { }
+    @Override public void info(Throwable thrown) { }
+    @Override public void info(String msg, Throwable thrown) { }
+    @Override public boolean isDebugEnabled() { return false; }
+    @Override public void setDebugEnabled(boolean enabled) { }
+    @Override public void debug(String msg, Object... args) { }
+    @Override public void debug(String msg, long value) { }
+    @Override public void debug(Throwable thrown) { }
+    @Override public void debug(String msg, Throwable thrown) { }
+    @Override public Logger getLogger(String name) { return this; }
+    @Override public void ignore(Throwable ignored) { }
 }

src/main/java/de/sldk/mc/prometheusexporter/MetricsController.java

@@ -57,8 +57,7 @@ public void handle(String target, Request baseRequest, HttpServletRequest reques
 
             baseRequest.setHandled(true);
         } catch (IOException e) {
-            exporter.getLogger().error("Failed to read server statistics");
-            e.printStackTrace();
+            exporter.getLogger().error("Failed to read server statistics", e);
             response.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
         }
     }

src/main/java/de/sldk/mc/prometheusexporter/PrometheusExporter.java

@@ -32,22 +32,23 @@ public class PrometheusExporter {
     @Inject
     @DefaultConfig(sharedRoot = false)
     private File config;
+
     @Inject
     @DefaultConfig(sharedRoot = false)
     private ConfigurationLoader<CommentedConfigurationNode> cfgMgr;
     private ConfigurationNode cfg;
 
-    private static PrometheusExporter instance;
-    public static PrometheusExporter getInstance() {
-        return PrometheusExporter.instance;
-    }
-
     @Inject
     private Logger logger;
     public Logger getLogger() {
         return this.logger;
     }
 
+    private static PrometheusExporter instance;
+    public static PrometheusExporter getInstance() {
+        return PrometheusExporter.instance;
+    }
+
     @Listener
     public void onPreinit(GamePreInitializationEvent event) {
         logger.info("Setting up config...");
@@ -69,17 +70,16 @@ public void onPreinit(GamePreInitializationEvent event) {
 
     @Listener
     public void onServerStarted(GameStartedServerEvent event) {
-        org.eclipse.jetty.util.log.Log.setLog(new JettyNullLogger());
         try {
             server = new Server(port);
 
             server.setHandler(new MetricsController(this));
             server.start();
 
-            getLogger().info("Started Prometheus metrics endpoint on port " + port);
+            logger.info("Started Prometheus metrics endpoint on port " + port);
 
         } catch (Exception e) {
-            getLogger().error("Could not start embedded Jetty server", e);
+            logger.error("Could not start embedded Jetty server", e);
         }
     }
 

src/main/resources/jetty-logging.properties

@@ -0,0 +1 @@
+org.eclipse.jetty.util.log.class=de.sldk.mc.prometheusexporter.JettyNullLogger