Yes it makes sense. Thanks for the fix and the clean solution! I have a concern, though, with the default for 1.9.x. See my comment in the pull request: https://github.com/jclouds/jclouds/pull/830#issuecomment-126439968
That (IMO) should be addressed before we release 1.9.1. El 30/7/2015 18:29, "Arvindn05" <arvind...@gmail.com> escribió: > I do have one pull request pending for 1.9.1. It fixes a security issue in > keystone. > > https://github.com/jclouds/jclouds/pull/830 > > Hoping it will make the release. > > Arvind > > > > > On Jul 30, 2015, at 9:11 AM, Andrew Gaul <g...@apache.org> wrote: > > > > We do not have a reproducible test case for the GSON issue so let's move > > forward without upgrading it. > > > >> On Wed, Jul 29, 2015 at 12:58:59PM +0200, Andrew Bayer wrote: > >> Alright - I'll see if there's any news on the GSON issue by tomorrow EU > >> time and then cut an RC in the morning here. > >> > >>> On Wed, Jul 29, 2015 at 12:12 PM, Ignasi Barrera <n...@apache.org> > wrote: > >>> > >>> I've just seen you've already merged it. Thx! > >>> > >>>> On 29 July 2015 at 12:12, Ignasi Barrera <n...@apache.org> wrote: > >>>> +1 to run the release, but please, can you kindly merge my AWS live > >>>> tests PR then? :) > >>>> > >>>>> On 29 July 2015 at 12:00, Andrew Bayer <andrew.ba...@gmail.com> > wrote: > >>>>> Being selfish, I'd really like to have this out by next week so I can > >>>>> upgrade to 1.9.1 without needing a SNAPSHOT, but if this is important > >>>>> enough, I can muddle on with the SNAPSHOT for now... > >>>>> > >>>>> A. > >>>>> > >>>>>> On Tue, Jul 28, 2015 at 5:48 PM, Andrew Gaul <g...@apache.org> > wrote: > >>>>>> > >>>>>> Would be nice to resolve the GSON 2.3 issue: > >>>>>> > >>>>>> https://issues.apache.org/jira/browse/JCLOUDS-974 > >>>>>> > >>>>>> I still do not undrestand this and will not have time to look at it > >>>>>> today. Can anyone else investigate? > >>>>>> > >>>>>>> On Tue, Jul 28, 2015 at 04:43:57PM +0200, Andrew Bayer wrote: > >>>>>>> So - what's left for 1.9.1? I'm ready to cut the release while > >>> Ignasi's > >>>>>>> traveling this week. > >>>>>>> > >>>>>>> A. > >>>>>> > >>>>>> -- > >>>>>> Andrew Gaul > >>>>>> http://gaul.org/ > > > > -- > > Andrew Gaul > > http://gaul.org/ >
