Fix connection error spamming (#847)

* Fix "pointers" for players never being used lol

* Added Platform Facet API to add Type pointers

* Remove duplicate

* Fix some connection things

* Make checkstyle happy?

proxy/src/main/java/com/velocitypowered/proxy/connection/client/ConnectedPlayer.java

@@ -706,6 +706,11 @@ public class ConnectedPlayer implements MinecraftConnectionAssociation, Player,
                   switch (status.getStatus()) {
                     // Impossible/nonsensical cases
                     case ALREADY_CONNECTED:
+                      logger.error("{}: already connected to {}",
+                          this,
+                          status.getAttemptedConnection().getServerInfo().getName()
+                      );
+                      break;
                     case CONNECTION_IN_PROGRESS:
                       // Fatal case
                     case CONNECTION_CANCELLED:
@@ -1179,12 +1184,8 @@ public class ConnectedPlayer implements MinecraftConnectionAssociation, Player,
             if (status != null && !status.isSuccessful()) {
               if (!status.isSafe()) {
                 handleConnectionException(status.getAttemptedConnection(), throwable, false);
-                return;
               }
             }
-            if (throwable != null) {
-              logger.error("Exception during connect; status = {}", status, throwable);
-            }
           }, connection.eventLoop())
           .thenApply(x -> x);
     }