Re: [aerogear-dev] AeroGear Android Push 0.1.1

Option 2 sounds like a plan to me

On 2014-05-06, Daniel Passos wrote: > Talking with Summers about what is the best options to solved that, we have > some options: > > 1) Catch the 302 and handle it inside of the rest provider. [1] > > 2) We can fix the IOException which is being thrown, throw the > HttpException correctly, and in the registrar check for 30x and then rerun > the operation [2] > > 3) We let the HttpException escape to the callback handler > > We don't like the option 1, because increases complexity in the > HttpRestProvider class and it hides the 30x messages from the developers > should they want to handle them. Furthermore, Summers argues that since > the registration code is being called often it would be better for the > developer to be aware of them and have to handle them in her code instead > of ours. In short, if there are redirects then the developer should be > made aware so she can update her URL and make fewer HTTP requests (this is > important on mobile for battery life, data caps, etc). > > [1] > https://github.com/aerogear/aerogear-android/blob/master/src/org/jboss/ae... > [2] > https://github.com/aerogear/aerogear-android/blob/master/src/org/jboss/ae... > > So, wdyt? > > > > On Tue, May 6, 2014 at 6:47 AM, Sebastien Blanc <scm.blanc(a)gmail.com&gt; wrote: > >> Hi ! >> I just gave it a test with the Push HelloWorld : >> >> - https://issues.jboss.org/browse/AGPUSH-627 is fixed , I'm getting a >> nice error notification on the device >> >> - https://issues.jboss.org/browse/AGDROID-214 -> I'm getting exactly the >> same issue as Matzew , looks like the redirect is not working. >> >> Sebi >> >> >> >> On Fri, May 2, 2014 at 5:06 PM, Daniel Passos <daniel(a)passos.me&gt; wrote: >> >>> We have found a bug[1] in the AeroGear Android Push 0.1. It was fixed and >>> the new version (0.1.1) was already sent to nexus[2]. >>> >>> We'd like to ship it to Maven central next wednesday. >>> >>> [1] https://issues.jboss.org/browse/AGDROID-214 >>> [2] >>> http://repository.jboss.org/nexus/content/repositories/jboss_releases_sta... >>> >>> >>> _______________________________________________ >>> aerogear-dev mailing list >>> aerogear-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/aerogear-dev >>> >> >> >> _______________________________________________ >> aerogear-dev mailing list >> aerogear-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/aerogear-dev >> > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev -- abstractj _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

>Phone:404 941 4698 >Java is my crack.