Add debug code

bukkit/pom.xml

@@ -165,7 +165,7 @@
         <dependency>
             <groupId>io.papermc.paper</groupId>
             <artifactId>paper-api</artifactId>
-            <version>1.19-R0.1-SNAPSHOT</version>
+            <version>1.19.4-R0.1-SNAPSHOT</version>
             <scope>provided</scope>
             <!-- Use our own newer api version -->
             <exclusions>

bukkit/src/main/java/com/github/games647/fastlogin/bukkit/listener/protocollib/ProtocolLibListener.java

@@ -250,7 +250,6 @@ public class ProtocolLibListener extends PacketAdapter {
             }
         }
 
-
         plugin.getLog().trace("GameProfile {} with {} connecting", sessionKey, username);
 
         packetEvent.getAsyncMarker().incrementProcessingDelay();

core/pom.xml

@@ -70,7 +70,7 @@
         <dependency>
             <groupId>com.zaxxer</groupId>
             <artifactId>HikariCP</artifactId>
-            <version>4.0.3</version>
+            <version>5.0.2-SNAPSHOT</version>
             <exclusions>
                 <!-- HikariCP uses an old version of this API that has a typo in the service interface -->
                 <!-- We will use the api provided by the jdk14 dependency -->

core/src/main/java/com/github/games647/fastlogin/core/storage/MySQLStorage.java

@@ -55,6 +55,9 @@ public class MySQLStorage extends SQLStorage {
         config.addDataSourceProperty("useSSL", useSSL);
         config.addDataSourceProperty("requireSSL", useSSL);
 
+        //enable leak detection of holding connections
+        config.setLeakDetectionThreshold(10_000);
+
         // adding paranoid, hides hostname, username, version and so
         // could be useful for hiding server details
         config.addDataSourceProperty("paranoid", true);
@@ -62,7 +65,7 @@ public class MySQLStorage extends SQLStorage {
         config.setJdbcUrl(JDBC_PROTOCOL + buildJDBCUrl(driver, host, port, database));
 
         // enable MySQL specific optimizations
-        addPerformanceProperties(config);
+//        addPerformanceProperties(config);
         return config;
     }
 

core/src/main/java/com/github/games647/fastlogin/core/storage/SQLStorage.java

@@ -143,11 +143,12 @@ public abstract class SQLStorage implements AuthStorage {
 
     @Override
     public void save(StoredProfile playerProfile) {
-        try (Connection con = dataSource.getConnection()) {
+        String uuid = playerProfile.getOptId().map(UUIDAdapter::toMojangId).orElse(null);
-            String uuid = playerProfile.getOptId().map(UUIDAdapter::toMojangId).orElse(null);
+        core.getPlugin().getLog().info("Before Lock");
-
+        synchronized (playerProfile) {
-            playerProfile.getSaveLock().lock();
+            core.getPlugin().getLog().info("Inside Lock - Before acquiring connection");
-            try {
+            try (Connection con = dataSource.getConnection()) {
+                core.getPlugin().getLog().info("Acquired connection");
                 if (playerProfile.isSaved()) {
                     try (PreparedStatement saveStmt = con.prepareStatement(UPDATE_PROFILE)) {
                         saveStmt.setString(1, uuid);
@@ -174,12 +175,14 @@ public abstract class SQLStorage implements AuthStorage {
                         }
                     }
                 }
-            } finally {
+            } catch (SQLException ex) {
-                playerProfile.getSaveLock().unlock();
+                core.getPlugin().getLog().error("Failed to save playerProfile {}", playerProfile, ex);
             }
-        } catch (SQLException ex) {
+
-            core.getPlugin().getLog().error("Failed to save playerProfile {}", playerProfile, ex);
+            core.getPlugin().getLog().info("Released connection");
         }
+
+        core.getPlugin().getLog().info("Released lock");
     }
 
     @Override