ok2c commented on code in PR #424:
URL: 
https://github.com/apache/httpcomponents-core/pull/424#discussion_r1312801174


##########
httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java:
##########
@@ -127,4 +133,32 @@ public void process(final HttpRequest request, final 
EntityDetails entity, final
         }
     }
 
+    /**
+     * Validates the presence and format of the Content-Type header for an 
OPTIONS request.
+     *
+     * <p>
+     * According to the HTTP protocol, an {@link Method#OPTIONS} request that 
contains content must also have

Review Comment:
   @reschke What you are saying makes no sense. @arturobernalg We cannot truly 
know what if `content-type` value is valid or not unless we maintain a list of 
vales we consider valid.   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org
For additional commands, e-mail: dev-h...@hc.apache.org

Reply via email to