The HTTP protocol version: it misses in AsyncApacheHttp5Client's response and hardcoded in Request.toString() (#2097)

* Missed the HTTP protocol version in HC5. The hardcoded protocol version in Request.toString()

* Fix formatting

Author: vitalijr2

core/src/main/java/feign/Request.java

@@ -290,7 +290,13 @@ public ProtocolVersion protocolVersion() {
   @Override
   public String toString() {
     final StringBuilder builder = new StringBuilder();
-    builder.append(httpMethod).append(' ').append(url).append(" HTTP/1.1\n");
+    builder
+        .append(httpMethod)
+        .append(' ')
+        .append(url)
+        .append(' ')
+        .append(protocolVersion)
+        .append('\n');
     for (final String field : headers.keySet()) {
       for (final String value : valuesOrEmpty(headers, field)) {
         builder.append(field).append(": ").append(value).append('\n');

hc5/src/main/java/feign/hc5/AsyncApacheHttp5Client.java

@@ -13,6 +13,8 @@
  */
 package feign.hc5;
 
+import static feign.Util.enumForName;
+
 import feign.*;
 import feign.Request.Options;
 import java.util.*;
@@ -179,6 +181,8 @@ Response toFeignResponse(SimpleHttpResponse httpResponse, Request request) {
     }
 
     return Response.builder()
+        .protocolVersion(
+            enumForName(Request.ProtocolVersion.class, httpResponse.getVersion().format()))
         .status(statusCode)
         .reason(reason)
         .headers(headers)