Removed invalid request version check from ResponseCachingPolicy

Author: ok2c

httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/ResponseCachingPolicy.java

@@ -90,14 +90,6 @@ public ResponseCachingPolicy(
      * @return {@code true} if response is cacheable
      */
     public boolean isResponseCacheable(final ResponseCacheControl cacheControl, final HttpRequest request, final HttpResponse response) {
-        final ProtocolVersion version = request.getVersion() != null ? request.getVersion() : HttpVersion.DEFAULT;
-        if (version.compareToVersion(HttpVersion.HTTP_1_1) > 0) {
-            if (LOG.isDebugEnabled()) {
-                LOG.debug("Protocol version {} is non-cacheable", version);
-            }
-            return false;
-        }
-
         // Presently only GET and HEAD methods are supported
         final String httpMethod = request.getMethod();
         if (!Method.GET.isSame(httpMethod) && !Method.HEAD.isSame(httpMethod)) {

httpclient5-cache/src/test/java/org/apache/hc/client5/http/impl/cache/TestResponseCachingPolicy.java

@@ -1000,4 +1000,20 @@ void testMustRevalidateWithAuthorizationIsCacheable() {
                 "Response with must-revalidate and Authorization header should be cacheable in shared cache.");
     }
 
+    @Test
+    void testHttp2_0HintCacheable() {
+        request = new BasicHttpRequest("GET", "/");
+        request.setVersion(HttpVersion.HTTP_2);
+        response = new BasicHttpResponse(HttpStatus.SC_OK, "OK");
+        Assertions.assertTrue(policy.isResponseCacheable(responseCacheControl, request, response));
+    }
+
+    @Test
+    void testHttp1_0HintCacheable() {
+        request = new BasicHttpRequest("GET", "/");
+        request.setVersion(HttpVersion.HTTP_1_0);
+        response = new BasicHttpResponse(HttpStatus.SC_OK, "OK");
+        Assertions.assertTrue(policy.isResponseCacheable(responseCacheControl, request, response));
+    }
+
 }