Handle nullable http-request path correctly

Author: TBlueF

common/src/main/java/de/bluecolored/bluemap/common/web/FileRequestHandler.java

@@ -65,6 +65,7 @@ public HttpResponse handle(HttpRequest request) {
 
     private HttpResponse generateResponse(HttpRequest request) throws IOException {
         String path = request.getPath();
+        if (path == null) return new HttpResponse(HttpStatusCode.BAD_REQUEST);
 
         // normalize path
         if (path.startsWith("/")) path = path.substring(1);

common/src/main/java/de/bluecolored/bluemap/common/web/RoutingRequestHandler.java

@@ -67,6 +67,7 @@ public void register(Pattern pattern, String replacementRoute, HttpRequestHandle
     @Override
     public HttpResponse handle(HttpRequest request) {
         String path = request.getPath();
+        if (path == null) return new HttpResponse(HttpStatusCode.BAD_REQUEST);
 
         // normalize path
         if (path.startsWith("/")) path = path.substring(1);

common/src/main/java/de/bluecolored/bluemap/common/web/http/HttpRequest.java

@@ -24,6 +24,8 @@
  */
 package de.bluecolored.bluemap.common.web.http;
 
+import org.jetbrains.annotations.Nullable;
+
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.InputStream;
@@ -218,7 +220,7 @@ public InputStream getDataStream() {
         return new ByteArrayInputStream(data);
     }
 
-    public String getPath() {
+    public @Nullable String getPath() {
         if (path == null) parseAddress();
         return path;
     }
@@ -232,7 +234,7 @@ public Map<String, String> getGETParams() {
         return getParams;
     }
 
-    public String getGETParamString() {
+    public @Nullable String getGETParamString() {
         if (getParamString == null) parseAddress();
         return getParamString;
     }
@@ -249,12 +251,15 @@ private void parseAddress() {
 
     private void parseGetParams() {
         Map<String, String> getParams = new HashMap<>();
-        for (String getParam : this.getGETParamString().split("&")){
-            if (getParam.isEmpty()) continue;
-            String[] kv = getParam.split("=", 2);
-            String key = kv[0];
-            String value = kv.length > 1 ? kv[1] : "";
-            getParams.put(key, value);
+        String getParamString = this.getGETParamString();
+        if (getParamString != null) {
+            for (String getParam : getParamString.split("&")) {
+                if (getParam.isEmpty()) continue;
+                String[] kv = getParam.split("=", 2);
+                String key = kv[0];
+                String value = kv.length > 1 ? kv[1] : "";
+                getParams.put(key, value);
+            }
         }
         this.getParams = getParams;
     }