This commit is contained in:
Xephi 2016-02-05 13:24:38 +01:00
parent e8a9b5c2e2
commit 4f76398ce8
15 changed files with 50 additions and 77 deletions

View File

@ -227,9 +227,7 @@ public class AuthMe extends JavaPlugin {
try { try {
setupDatabase(); setupDatabase();
} catch (Exception e) { } catch (Exception e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace(e.getMessage() + "\nFatal error occurred during database connection! Authme initialization ABORTED!" , e);
ConsoleLogger.showError(e.getMessage());
ConsoleLogger.showError("Fatal error occurred during database connection! Authme initialization ABORTED!");
stopOrUnload(); stopOrUnload();
return; return;
} }
@ -452,9 +450,8 @@ public class AuthMe extends JavaPlugin {
settings = new Settings(this); settings = new Settings(this);
Settings.reload(); Settings.reload();
} catch (Exception e) { } catch (Exception e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Can't load the configuration file... Something went wrong. "
ConsoleLogger.showError("Can't load the configuration file... Something went wrong. " + "To avoid security issues the server will shut down!", e);
+ "To avoid security issues the server will shut down!");
server.shutdown(); server.shutdown();
return true; return true;
} }
@ -516,8 +513,7 @@ public class AuthMe extends JavaPlugin {
ConsoleLogger.info("Metrics started successfully!"); ConsoleLogger.info("Metrics started successfully!");
} catch (Exception e) { } catch (Exception e) {
// Failed to submit the metrics data // Failed to submit the metrics data
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Can't start Metrics! The plugin will work anyway...", e);
ConsoleLogger.showError("Can't start Metrics! The plugin will work anyway...");
} }
} }

View File

@ -74,10 +74,11 @@ public final class ConsoleLogger {
* *
* @param ex Exception * @param ex Exception
*/ */
public static void writeStackTrace(Exception ex) { public static void writeStackTrace(String message , Throwable ex) {
if (!Settings.useLogging) { if (!Settings.useLogging) {
return; return;
} }
writeLog(message);
writeLog(Throwables.getStackTraceAsString(ex)); writeLog(Throwables.getStackTraceAsString(ex));
} }
} }

View File

@ -26,8 +26,7 @@ public class ReloadCommand implements ExecutableCommand {
plugin.setupDatabase(); plugin.setupDatabase();
} catch (Exception e) { } catch (Exception e) {
sender.sendMessage("Error occurred during reload of AuthMe: aborting"); sender.sendMessage("Error occurred during reload of AuthMe: aborting");
ConsoleLogger.showError("Fatal error occurred! AuthMe instance ABORTED!"); ConsoleLogger.writeStackTrace("Fatal error occurred! AuthMe instance ABORTED!", e);
ConsoleLogger.writeStackTrace(e);
plugin.stopOrUnload(); plugin.stopOrUnload();
} }

View File

@ -33,8 +33,7 @@ public class RoyalAuthConverter implements Converter {
PlayerAuth auth = new PlayerAuth(name, ra.getHash(), "127.0.0.1", ra.getLastLogin(), "your@email.com", o.getName()); PlayerAuth auth = new PlayerAuth(name, ra.getHash(), "127.0.0.1", ra.getLastLogin(), "your@email.com", o.getName());
data.saveAuth(auth); data.saveAuth(auth);
} catch (Exception e) { } catch (Exception e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Error while trying to import " + o.getName() + " RoyalAuth datas", e);
ConsoleLogger.showError("Error while trying to import " + o.getName() + " RoyalAuth datas");
} }
} }
} }

View File

@ -53,7 +53,7 @@ class vAuthFileReader {
} }
scanner.close(); scanner.close();
} catch (Exception e) { } catch (Exception e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Error while trying to import some vAuth datas", e);
} }
} }

View File

@ -172,7 +172,7 @@ public class CacheDataSource implements DataSource {
exec.shutdown(); exec.shutdown();
exec.awaitTermination(8, TimeUnit.SECONDS); exec.awaitTermination(8, TimeUnit.SECONDS);
} catch (InterruptedException e) { } catch (InterruptedException e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Error while closing CacheDataSource.", e);
} }
source.close(); source.close();
} }

View File

@ -257,8 +257,7 @@ public class MySQL implements DataSource {
ResultSet rs = pst.executeQuery(); ResultSet rs = pst.executeQuery();
return rs.next(); return rs.next();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return false; return false;
} }
@ -276,8 +275,7 @@ public class MySQL implements DataSource {
!columnSalt.isEmpty() ? rs.getString(columnSalt) : null); !columnSalt.isEmpty() ? rs.getString(columnSalt) : null);
} }
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return null; return null;
} }
@ -322,8 +320,7 @@ public class MySQL implements DataSource {
} }
} }
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
return null; return null;
} }
return pAuth; return pAuth;
@ -525,8 +522,7 @@ public class MySQL implements DataSource {
} }
return true; return true;
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return false; return false;
} }
@ -589,8 +585,7 @@ public class MySQL implements DataSource {
} }
return true; return true;
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return false; return false;
} }
@ -623,8 +618,7 @@ public class MySQL implements DataSource {
pst.setLong(1, until); pst.setLong(1, until);
result = pst.executeUpdate(); result = pst.executeUpdate();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return result; return result;
} }
@ -647,8 +641,7 @@ public class MySQL implements DataSource {
st.executeUpdate(); st.executeUpdate();
st.close(); st.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return list; return list;
} }
@ -680,8 +673,7 @@ public class MySQL implements DataSource {
pst.executeUpdate(); pst.executeUpdate();
return true; return true;
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return false; return false;
} }
@ -702,8 +694,7 @@ public class MySQL implements DataSource {
pst.close(); pst.close();
return true; return true;
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return false; return false;
} }
@ -722,8 +713,7 @@ public class MySQL implements DataSource {
rs.close(); rs.close();
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return countIp; return countIp;
} }
@ -739,8 +729,7 @@ public class MySQL implements DataSource {
pst.close(); pst.close();
return true; return true;
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return false; return false;
} }
@ -750,9 +739,8 @@ public class MySQL implements DataSource {
try { try {
reloadArguments(); reloadArguments();
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.showError("Can't reconnect to MySQL database... Please check your MySQL configuration!"); ConsoleLogger.showError("Can't reconnect to MySQL database... Please check your MySQL configuration!");
ConsoleLogger.writeStackTrace(ex);
AuthMe.getInstance().stopOrUnload(); AuthMe.getInstance().stopOrUnload();
} }
} }
@ -778,8 +766,7 @@ public class MySQL implements DataSource {
rs.close(); rs.close();
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return result; return result;
} }
@ -798,8 +785,7 @@ public class MySQL implements DataSource {
rs.close(); rs.close();
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return result; return result;
} }
@ -818,8 +804,7 @@ public class MySQL implements DataSource {
rs.close(); rs.close();
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return countEmail; return countEmail;
} }
@ -834,8 +819,7 @@ public class MySQL implements DataSource {
} }
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
} }
@ -854,8 +838,7 @@ public class MySQL implements DataSource {
ResultSet rs = pst.executeQuery(); ResultSet rs = pst.executeQuery();
isLogged = rs.next() && (rs.getInt(columnLogged) == 1); isLogged = rs.next() && (rs.getInt(columnLogged) == 1);
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return isLogged; return isLogged;
} }
@ -870,8 +853,7 @@ public class MySQL implements DataSource {
pst.executeUpdate(); pst.executeUpdate();
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
} }
@ -885,8 +867,7 @@ public class MySQL implements DataSource {
pst.executeUpdate(); pst.executeUpdate();
pst.close(); pst.close();
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
} }
@ -900,8 +881,7 @@ public class MySQL implements DataSource {
pst.executeUpdate(); pst.executeUpdate();
pst.close(); pst.close();
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
} }
@ -917,8 +897,7 @@ public class MySQL implements DataSource {
rs.close(); rs.close();
st.close(); st.close();
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return result; return result;
} }
@ -932,8 +911,7 @@ public class MySQL implements DataSource {
pst.setString(2, oldOne); pst.setString(2, oldOne);
pst.executeUpdate(); pst.executeUpdate();
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
} }
@ -978,8 +956,7 @@ public class MySQL implements DataSource {
rs.close(); rs.close();
st.close(); st.close();
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return auths; return auths;
} }
@ -1022,8 +999,7 @@ public class MySQL implements DataSource {
auths.add(pAuth); auths.add(pAuth);
} }
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} }
return auths; return auths;
} }

View File

@ -170,8 +170,7 @@ public class SQLite implements DataSource {
!columnSalt.isEmpty() ? rs.getString(columnSalt) : null); !columnSalt.isEmpty() ? rs.getString(columnSalt) : null);
} }
} catch (SQLException ex) { } catch (SQLException ex) {
ConsoleLogger.showError(ex.getMessage()); ConsoleLogger.writeStackTrace(ex.getMessage(), ex);
ConsoleLogger.writeStackTrace(ex);
} finally { } finally {
close(rs); close(rs);
close(pst); close(pst);

View File

@ -135,8 +135,7 @@ public class ModuleManager {
} }
} catch (Exception ex) { } catch (Exception ex) {
ConsoleLogger.writeStackTrace(ex); ConsoleLogger.writeStackTrace("Cannot load " + pathToJar.getName() + " jar file !", ex);
ConsoleLogger.showError("Cannot load " + pathToJar.getName() + " jar file !");
} finally { } finally {
try { try {
if (jarFile != null) { if (jarFile != null) {

View File

@ -49,6 +49,13 @@ public class ProcessSyncronousPlayerLogout implements Runnable {
player.sendPluginMessage(plugin, "BungeeCord", out.toByteArray()); player.sendPluginMessage(plugin, "BungeeCord", out.toByteArray());
} }
protected void restoreSpeedEffect() {
if (Settings.isRemoveSpeedEnabled) {
player.setWalkSpeed(0.0F);
player.setFlySpeed(0.0F);
}
}
/** /**
* Method run. * Method run.
* *
@ -77,6 +84,7 @@ public class ProcessSyncronousPlayerLogout implements Runnable {
if (Settings.applyBlindEffect) if (Settings.applyBlindEffect)
player.addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS, Settings.getRegistrationTimeout * 20, 2)); player.addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS, Settings.getRegistrationTimeout * 20, 2));
player.setOp(false); player.setOp(false);
restoreSpeedEffect();
// Player is now logout... Time to fire event ! // Player is now logout... Time to fire event !
Bukkit.getServer().getPluginManager().callEvent(new LogoutEvent(player)); Bukkit.getServer().getPluginManager().callEvent(new LogoutEvent(player));
if (Settings.bungee) if (Settings.bungee)

View File

@ -81,8 +81,7 @@ public class AsyncRegister {
passwordRegister(); passwordRegister();
} }
} catch (Exception e) { } catch (Exception e) {
ConsoleLogger.showError(e.getMessage()); ConsoleLogger.writeStackTrace(e.getMessage(), e);
ConsoleLogger.writeStackTrace(e);
m.send(player, MessageKey.ERROR); m.send(player, MessageKey.ERROR);
} }
} }

View File

@ -77,8 +77,7 @@ public abstract class CustomConfiguration extends YamlConfiguration {
return true; return true;
} }
} catch (Exception e) { } catch (Exception e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Failed to load config from JAR", e);
ConsoleLogger.showError("Failed to load config from JAR");
} }
} }
return false; return false;

View File

@ -313,8 +313,7 @@ public final class Settings {
try { try {
return Files.toString(EMAIL_FILE, Charsets.UTF_8); return Files.toString(EMAIL_FILE, Charsets.UTF_8);
} catch (IOException e) { } catch (IOException e) {
ConsoleLogger.showError("Error loading email text: " + StringUtils.formatException(e)); ConsoleLogger.writeStackTrace("Error loading email text: " + StringUtils.formatException(e), e);
ConsoleLogger.writeStackTrace(e);
return ""; return "";
} }
} }

View File

@ -129,8 +129,7 @@ public class NewSetting {
writer.flush(); writer.flush();
writer.close(); writer.close();
} catch (IOException e) { } catch (IOException e) {
ConsoleLogger.showError("Could not save config file - " + StringUtils.formatException(e)); ConsoleLogger.writeStackTrace("Could not save config file - " + StringUtils.formatException(e), e);
ConsoleLogger.writeStackTrace(e);
} }
} }

View File

@ -36,7 +36,7 @@ public class GeoLiteAPI {
plugin.getLogger().info(LICENSE); plugin.getLogger().info(LICENSE);
return true; return true;
} catch (IOException e) { } catch (IOException e) {
// TODO ljacqu 20151123: Log the exception instead of just swallowing it ConsoleLogger.writeStackTrace("Could not find/download GeoLiteAPI", e);
return false; return false;
} }
} }
@ -63,7 +63,7 @@ public class GeoLiteAPI {
output.close(); output.close();
input.close(); input.close();
} catch (IOException e) { } catch (IOException e) {
ConsoleLogger.writeStackTrace(e); ConsoleLogger.writeStackTrace("Could not download GeoLiteAPI", e);
} }
} }
}); });