For the record: https://issues.apache.org/jira/browse/JCLOUDS-405
On 12 December 2013 16:23, Ignasi Barrera <[email protected]> wrote: > Verified with the Charles proxy that the fix works. Will send the PR > as soon as tests finish :) > > On 12 December 2013 15:58, Ignasi Barrera <[email protected]> wrote: >> Trying it right now... Will send a PR if it works >> >> On 12 December 2013 15:36, Ignasi Barrera <[email protected]> wrote: >>> I found this comment [1] in the Jersey issue, explaining that in SSL >>> there is an HTTPS URL connection wrapping the actual HttpURLConnection >>> where the method should be set. >>> Could you do a quick test and add the appropriate bits from the code >>> in the Jersey issue and see if that does the trick for HTTPS >>> connections? >>> >>> >>> [1] >>> https://java.net/jira/browse/JERSEY-639?focusedCommentId=370981&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#action_370981 >>> >>> On 12 December 2013 15:30, Everett Toews <[email protected]> >>> wrote: >>>> Yes. The request is going to https://ord.queues.api.rackspacecloud.com. >>>> >>>> There's some special handling for HTTPS [1] but the part for setting the >>>> request method seems to be common for both HTTP and HTTPS. >>>> >>>> What did you have in mind? >>>> >>>> Everett >>>> >>>> [1] >>>> https://github.com/jclouds/jclouds/blob/master/core/src/main/java/org/jclouds/http/internal/JavaUrlHttpCommandExecutorService.java#L158 >>>> >>>> >>>> On Dec 12, 2013, at 2:29 AM, Ignasi Barrera wrote: >>>> >>>>> This is strange. The reflection fix is the same used in the Jersey >>>>> client [1], and it is supposed to work. >>>>> Looking at the jclouds code, there is no special handling for HTTPS >>>>> connections. Everett, are you using an HTTPS one? >>>>> >>>>> >>>>> >>>>> [1] https://java.net/jira/browse/JERSEY-639 >>>>> >>>>> On 12 December 2013 09:09, Andrew Phillips <[email protected]> wrote: >>>>>>> I know that HttpURLConnection.setRequest() [2] doesn't allow PATCH. I >>>>>>> also know that we work around this in jclouds by setting the field by >>>>>>> reflection [3]. >>>>>> >>>>>> >>>>>> Have you been able to try this with the apachehc [1] driver? Does that >>>>>> make >>>>>> any difference? >>>>>> >>>>>> ap >>>>>> >>>>>> [1] https://github.com/jclouds/jclouds/tree/master/drivers/apachehc >>>>
