On 07/03/18 08:36, Andrej Golovnin wrote:
Hi Chris,
The JDK HTTP Client has:
`HttpRequest.Builder::method(String method, BodyPublisher publisher)`
,so it is currently possible to use the `PATCH` method.
Is `PATCH` sufficiently popular to warrant its own self-named
method in the request builder?
I don't know how popular is it. We have just started to use it with
one of our services and discovered that we cannot use it with
HttpURLConnection at all. And I thought when we add the support for
PATCH to HttpURLConnection, then it would be nice to have it in HTTP
Client too. When you decided not to add it to HTTP Client, then it is
OK for me too. But I think that HttpURLConnection should be changed.
I'm not opposed adding support for PATCH to HttpURLConnection.
You will need to add a test, and I think a CSR to add it to
list of supported methods.
-Chris.
