Merge pull request #9935 from jaikiran/qk-9882

Fix NullPointerException(s) when dealing with remote and local address returned by HTTP request

Author: gsmet

extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/attribute/LocalIPAttribute.java

@@ -20,6 +20,9 @@ private LocalIPAttribute() {
     @Override
     public String readAttribute(final RoutingContext exchange) {
         SocketAddress localAddress = exchange.request().localAddress();
+        if (localAddress == null) {
+            return null;
+        }
         return localAddress.host();
     }
 

extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/attribute/LocalPortAttribute.java

@@ -1,5 +1,6 @@
 package io.quarkus.vertx.http.runtime.attribute;
 
+import io.vertx.core.net.SocketAddress;
 import io.vertx.ext.web.RoutingContext;
 
 /**
@@ -19,7 +20,11 @@ private LocalPortAttribute() {
 
     @Override
     public String readAttribute(final RoutingContext exchange) {
-        return Integer.toString(exchange.request().localAddress().port());
+        final SocketAddress localAddr = exchange.request().localAddress();
+        if (localAddr == null) {
+            return null;
+        }
+        return Integer.toString(localAddr.port());
     }
 
     @Override

extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/attribute/RemoteHostAttribute.java

@@ -1,5 +1,6 @@
 package io.quarkus.vertx.http.runtime.attribute;
 
+import io.vertx.core.net.SocketAddress;
 import io.vertx.ext.web.RoutingContext;
 
 /**
@@ -19,7 +20,11 @@ private RemoteHostAttribute() {
 
     @Override
     public String readAttribute(final RoutingContext exchange) {
-        return exchange.request().remoteAddress().host();
+        final SocketAddress remoteAddr = exchange.request().remoteAddress();
+        if (remoteAddr == null) {
+            return null;
+        }
+        return remoteAddr.host();
     }
 
     @Override

extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/attribute/RemoteIPAttribute.java

@@ -21,6 +21,9 @@ private RemoteIPAttribute() {
     @Override
     public String readAttribute(final RoutingContext exchange) {
         final SocketAddress sourceAddress = exchange.request().remoteAddress();
+        if (sourceAddress == null) {
+            return null;
+        }
         return sourceAddress.host();
     }