Follow the Common Log Format in access logs for HTTP version when 'H' is used

Fixes: quarkusio#48122

Author: geoand

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

@@ -2,7 +2,6 @@
 
 import io.quarkus.vertx.http.runtime.filters.OriginalRequestContext;
 import io.vertx.core.http.HttpMethod;
-import io.vertx.core.http.HttpVersion;
 import io.vertx.ext.web.RoutingContext;
 
 /**
@@ -40,23 +39,7 @@ public String readAttribute(final RoutingContext exchange) {
             uri = exchange.request().uri();
         }
 
-        return httpMethod + " " + uri + " " + getHttpVersionStr(exchange.request().version());
-    }
-
-    private static String getHttpVersionStr(HttpVersion version) {
-        // best effort to try and infer the HTTP version from
-        // any "unknown" enum value
-        return switch (version) {
-            case HTTP_1_0 -> "HTTP/1.0";
-            case HTTP_1_1 -> "HTTP/1.1";
-            case HTTP_2 -> "HTTP/2";
-            default ->
-                // best effort to try and infer the HTTP version from
-                // any "unknown" enum value
-                    version.name()
-                            .replace("HTTP_", "HTTP/")
-                            .replace("_", ".");
-        };
+        return httpMethod + " " + uri + " " + RequestProtocolAttribute.getHttpVersionStr(exchange.request().version());
     }
 
     @Override

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

@@ -1,5 +1,6 @@
 package io.quarkus.vertx.http.runtime.attribute;
 
+import io.vertx.core.http.HttpVersion;
 import io.vertx.ext.web.RoutingContext;
 
 /**
@@ -19,14 +20,30 @@ private RequestProtocolAttribute() {
 
     @Override
     public String readAttribute(final RoutingContext exchange) {
-        return exchange.request().version().name();
+        return getHttpVersionStr(exchange.request().version());
     }
 
     @Override
     public void writeAttribute(final RoutingContext exchange, final String newValue) throws ReadOnlyAttributeException {
         throw new ReadOnlyAttributeException("Request getProtocol", newValue);
     }
 
+    static String getHttpVersionStr(HttpVersion version) {
+        // best effort to try and infer the HTTP version from
+        // any "unknown" enum value
+        return switch (version) {
+            case HTTP_1_0 -> "HTTP/1.0";
+            case HTTP_1_1 -> "HTTP/1.1";
+            case HTTP_2 -> "HTTP/2";
+            default ->
+                // best effort to try and infer the HTTP version from
+                // any "unknown" enum value
+                version.name()
+                        .replace("HTTP_", "HTTP/")
+                        .replace("_", ".");
+        };
+    }
+
     public static final class Builder implements ExchangeAttributeBuilder {
 
         @Override