<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 6, 2014 at 4:54 PM, Summers Pittman <span dir="ltr">&lt;<a href="mailto:supittma@redhat.com" target="_blank">supittma@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Tue 06 May 2014 10:42:10 AM EDT, Bruno Oliveira wrote:<br>
&gt; Option 2 sounds like a plan to me<br>
&gt;<br>
<br>
</div>Sweet.<br>
<br>
RestProvider throws httpexception (status:302)<br>
Registrar catches it and logs a warning.<br>
Retries with new location.<br>
if OK then onSuccess<br>
else then onError.<br>
<br>
IE We will only follow ONE redirect.<br></blockquote><div><br></div><div>yes :) no need to join a redirect party </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div><div class="h5"><br>
&gt; On 2014-05-06, Daniel Passos wrote:<br>
&gt;&gt; Talking with Summers about what is the best options to solved that, we have<br>
&gt;&gt; some options:<br>
&gt;&gt;<br>
&gt;&gt; 1) Catch the 302 and handle it inside of the rest provider. [1]<br>
&gt;&gt;<br>
&gt;&gt; 2) We can fix the IOException which is being thrown, throw the<br>
&gt;&gt; HttpException correctly, and in the registrar check for 30x and then rerun<br>
&gt;&gt; the operation [2]<br>
&gt;&gt;<br>
&gt;&gt; 3) We let the HttpException escape to the callback handler<br>
&gt;&gt;<br>
&gt;&gt; We don&#39;t like the option 1, because increases complexity in the<br>
&gt;&gt; HttpRestProvider class and it hides the 30x messages from the developers<br>
&gt;&gt; should they want to handle them.  Furthermore, Summers argues that since<br>
&gt;&gt; the registration code is being called often it would be better for the<br>
&gt;&gt; developer to be aware of them and have to handle them in her code instead<br>
&gt;&gt; of ours.  In short, if there are redirects then the developer should be<br>
&gt;&gt; made aware so she can update her URL and make fewer HTTP requests (this is<br>
&gt;&gt; important on mobile for battery life, data caps, etc).<br>
&gt;&gt;<br>
&gt;&gt; [1]<br>
&gt;&gt; <a href="https://github.com/aerogear/aerogear-android/blob/master/src/org/jboss/aerogear/android/impl/http/HttpRestProvider.java#L256" target="_blank">https://github.com/aerogear/aerogear-android/blob/master/src/org/jboss/aerogear/android/impl/http/HttpRestProvider.java#L256</a><br>

&gt;&gt; [2]<br>
&gt;&gt; <a href="https://github.com/aerogear/aerogear-android/blob/master/src/org/jboss/aerogear/android/impl/unifiedpush/AeroGearGCMPushRegistrar.java#L139-L144" target="_blank">https://github.com/aerogear/aerogear-android/blob/master/src/org/jboss/aerogear/android/impl/unifiedpush/AeroGearGCMPushRegistrar.java#L139-L144</a><br>

&gt;&gt;<br>
&gt;&gt; So, wdyt?<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Tue, May 6, 2014 at 6:47 AM, Sebastien Blanc &lt;<a href="mailto:scm.blanc@gmail.com">scm.blanc@gmail.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt;&gt; Hi !<br>
&gt;&gt;&gt; I just gave it a test with the Push HelloWorld :<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; - <a href="https://issues.jboss.org/browse/AGPUSH-627" target="_blank">https://issues.jboss.org/browse/AGPUSH-627</a> is fixed , I&#39;m getting a<br>
&gt;&gt;&gt; nice error notification on the device<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; - <a href="https://issues.jboss.org/browse/AGDROID-214" target="_blank">https://issues.jboss.org/browse/AGDROID-214</a> -&gt; I&#39;m getting exactly the<br>
&gt;&gt;&gt; same issue as Matzew , looks like the redirect is not working.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Sebi<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On Fri, May 2, 2014 at 5:06 PM, Daniel Passos &lt;<a href="mailto:daniel@passos.me">daniel@passos.me</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; We have found a bug[1] in the AeroGear Android Push 0.1. It was fixed and<br>
&gt;&gt;&gt;&gt; the new version (0.1.1) was already sent to nexus[2].<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; We&#39;d like to ship it to Maven central next wednesday.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; [1] <a href="https://issues.jboss.org/browse/AGDROID-214" target="_blank">https://issues.jboss.org/browse/AGDROID-214</a><br>
&gt;&gt;&gt;&gt; [2]<br>
&gt;&gt;&gt;&gt; <a href="http://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-3139/" target="_blank">http://repository.jboss.org/nexus/content/repositories/jboss_releases_staging_profile-3139/</a><br>

&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; _______________________________________________<br>
&gt;&gt;&gt;&gt; aerogear-dev mailing list<br>
&gt;&gt;&gt;&gt; <a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
&gt;&gt;&gt;&gt; <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; _______________________________________________<br>
&gt;&gt;&gt; aerogear-dev mailing list<br>
&gt;&gt;&gt; <a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
&gt;&gt;&gt; <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
&gt;&gt;&gt;<br>
&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; aerogear-dev mailing list<br>
&gt;&gt; <a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
&gt;&gt; <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt;<br>
&gt; abstractj<br>
&gt; _______________________________________________<br>
&gt; aerogear-dev mailing list<br>
&gt; <a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
&gt; <a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
<br>
<br>
<br>
</div></div>--<br>
Summers Pittman<br>
&gt;&gt;Phone:404 941 4698<br>
&gt;&gt;Java is my crack.<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>
sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a>
</div></div>