remove @NonNull, revert reindentation

api/src/main/java/com/velocitypowered/api/event/connection/PostLoginEvent.java

@@ -2,7 +2,6 @@ package com.velocitypowered.api.event.connection;
 
 import com.google.common.base.Preconditions;
 import com.velocitypowered.api.proxy.Player;
-import org.checkerframework.checker.nullness.qual.NonNull;
 
 /**
  * This event is fired once the player has been successfully authenticated and
@@ -12,7 +11,7 @@ public class PostLoginEvent {
 
     private final Player player;
 
-    public PostLoginEvent(@NonNull Player player) {
+    public PostLoginEvent(Player player) {
         this.player = Preconditions.checkNotNull(player, "player");
     }
 

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

@@ -42,8 +42,8 @@ import java.util.concurrent.ThreadLocalRandom;
 public class LoginSessionHandler implements MinecraftSessionHandler {
 
     private static final Logger logger = LogManager.getLogger(LoginSessionHandler.class);
-    private static final String MOJANG_SERVER_AUTH_URL
-            = "https://sessionserver.mojang.com/session/minecraft/hasJoined?username=%s&serverId=%s&ip=%s";
+    private static final String MOJANG_SERVER_AUTH_URL =
+            "https://sessionserver.mojang.com/session/minecraft/hasJoined?username=%s&serverId=%s&ip=%s";
 
     private final VelocityServer server;
     private final MinecraftConnection inbound;
@@ -195,26 +195,26 @@ public class LoginSessionHandler implements MinecraftSessionHandler {
                     apiInbound.getVirtualHost().orElse(null));
 
             return server.getEventManager().fire(new PermissionsSetupEvent(player, ConnectedPlayer.DEFAULT_PERMISSIONS))
-                    .thenCompose(event -> {
-                        // wait for permissions to load, then set the players permission function
-                        player.setPermissionFunction(event.createFunction(player));
-                        // then call & wait for the login event
-                        return server.getEventManager().fire(new LoginEvent(player));
-                    })
-                    // then complete the connection
-                    .thenAcceptAsync(event -> {
-                        if (inbound.isClosed()) {
-                            // The player was disconnected
-                            return;
-                        }
-                        if (!event.getResult().isAllowed()) {
-                            // The component is guaranteed to be provided if the connection was denied.
-                            inbound.closeWith(Disconnect.create(event.getResult().getReason().get()));
-                            return;
-                        }
+                        .thenCompose(event -> {
+                            // wait for permissions to load, then set the players permission function
+                            player.setPermissionFunction(event.createFunction(player));
+                            // then call & wait for the login event
+                            return server.getEventManager().fire(new LoginEvent(player));
+                        })
+                        // then complete the connection
+                        .thenAcceptAsync(event -> {
+                            if (inbound.isClosed()) {
+                                // The player was disconnected
+                                return;
+                            }
+                            if (!event.getResult().isAllowed()) {
+                                // The component is guaranteed to be provided if the connection was denied.
+                                inbound.closeWith(Disconnect.create(event.getResult().getReason().get()));
+                                return;
+                            }
 
-                        handleProxyLogin(player);
-                    }, inbound.getChannel().eventLoop());
+                            handleProxyLogin(player);
+                        }, inbound.getChannel().eventLoop());
         });
 
     }