hg: jdk8/tl/jdk: 7157360: HttpURLConnection: HTTP method DELETE doesn't support output

2013-06-22 Thread chris . hegarty
Changeset: fd050ba1cf72 Author:arieber Date: 2013-06-22 08:20 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fd050ba1cf72 7157360: HttpURLConnection: HTTP method DELETE doesn't support output Reviewed-by: chegar ! src/share/classes/sun/net/www/http/PosterOutputStream

Re: 7157360: HttpURLConnection: HTTP method DELETE doesn't support output

2013-06-21 Thread Chris Hegarty
On 21/06/2013 14:55, Andreas Rieber wrote: Hi Chris, actually from the HTTP spec any request on the server side should handle a request body even if not used or required by the request method. On You are correct Andreas. I meant 'harmless' from the stand-point of compatibility. Of course, we

Re: 7157360: HttpURLConnection: HTTP method DELETE doesn't support output

2013-06-21 Thread Andreas Rieber
Hi Chris, actually from the HTTP spec any request on the server side should handle a request body even if not used or required by the request method. On the client side they say the TRACE method must not have one. For all others its kind of open. I remember that we used request body in GET and

Re: 7157360: HttpURLConnection: HTTP method DELETE doesn't support output

2013-06-21 Thread Chris Hegarty
Hi Andreas, Your changes look fine to me. Strange to include a body in a DELETE request, but seems harmless. I noticed that the javafx issue is no longer dependent on this, but still worth fixing. I can sponsor this change into jdk8 for you. -Chris. On 20/06/2013 23:04, Andreas Rieber wro

7157360: HttpURLConnection: HTTP method DELETE doesn't support output

2013-06-20 Thread Andreas Rieber
Hi, here the next issue i would need a sponsor for. Also a very small one. Bug: http://bugs.sun.com/view_bug.do?bug_id=7157360 Looked straight forward to me for the DELETE method, but from last issue i still have the HTTP/1.1 spec open. Here the relevant parts: >> 4.3 Message Body ... The pres