jclouds-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <ignasi.barr...@gmail.com>
Subject Re: INTERNAL_SERVER_ERROR on getting versions of a vcloud server
Date Wed, 01 Jan 2014 15:39:35 GMT
I think it is a safe patch. According to the spec [1], not sending the
Accept header means the client accepts all media types, and the */* value
for the header means the same.

I also added that in the OkHttp driver pull request [2]. Nirmal, could you
open a pull request, as suggested by Andrew, with a method similar to the
one I added in my pull request, so the behavior can be overridden/extended
if needed?


[1] http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html
El 01/01/2014 16:17, "Andrew Phillips" <andrewp@apache.org> escribió:

> I think this is of no harm, what do you think?
> I think the easiest way to check this is to open a pull request and see if
> we get any test failures ;-) If everything looks OK and there are no
> concerns from committers, we could consider this.
> But unless "*/*" is the 'standard' default for the 'Accept' header if
> nothing else is specified, I suspect the correct fix for this will be to
> ensure that the vcloud provider sets this property appropriately.
> Have you been able to try a fix along those lines (see e.g. [1])
> ap
> [1] https://github.com/jclouds/jclouds/blob/master/apis/
> vcloud/src/main/java/org/jclouds/vcloud/internal/VCloudLoginApi.java#L42

View raw message