From 7d5e5deb248c12277f059e0506a20fa62fd18a4d Mon Sep 17 00:00:00 2001 From: asofold Date: Tue, 12 Feb 2013 22:58:03 +0100 Subject: [PATCH] Adjust Metrics. 1. Removed setting timeouts explicitly. 2. Schedule logging the error before throwing the exception. --- .../java/fr/neatmonster/nocheatplus/metrics/Metrics.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/metrics/Metrics.java b/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/metrics/Metrics.java index a92b181d..2ecbcb37 100644 --- a/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/metrics/Metrics.java +++ b/NCPPlugin/src/main/java/fr/neatmonster/nocheatplus/metrics/Metrics.java @@ -34,6 +34,8 @@ import org.bukkit.plugin.Plugin; import org.bukkit.plugin.PluginDescriptionFile; import org.bukkit.scheduler.BukkitTask; +import fr.neatmonster.nocheatplus.logging.LogUtil; + import java.io.BufferedReader; import java.io.File; import java.io.IOException; @@ -432,9 +434,9 @@ public class Metrics { } else { connection = url.openConnection(); } - // NCP: Set timeouts. - connection.setConnectTimeout(8000); - connection.setReadTimeout(16000); +// // NCP: Set timeouts. +// connection.setConnectTimeout(8000); +// connection.setReadTimeout(16000); connection.setDoOutput(true); @@ -462,6 +464,7 @@ public class Metrics { } if (response == null || response.startsWith("ERR")) { + LogUtil.scheduleLogWarning("[NoCheatPlus] Metrics error: " + response); throw new IOException(response); //Throw the exception } else { // Is this the first update this hour?