[aerogear-dev] AGDroid-Push:4.0.1 release (was Re: FCM: Cordova/Android bug ?)

https://issues.jboss.org/browse/AGDROID-560 On Mon, Jun 20, 2016 at 1:27 PM, Matthias Wessendorf <matzew(a)apache.org&gt; wrote: > I have seen that with a native app now too. > > I will create a JIRA > > On Mon, Jun 20, 2016 at 11:01 AM, Oleg Matskiv <omatskiv(a)redhat.com&gt; > wrote: > >> And instanceId.getToken() indeed returned null on first registration. >> So native unified-push-helloworld app [1] can be used to reproduce this >> issue. >> >> [1] https://github.com/jboss-mobile/unified-push-helloworld >> >> On Mon, Jun 20, 2016 at 10:40 AM, Oleg Matskiv <omatskiv(a)redhat.com&gt; >> wrote: >> >>> Hi, >>> >>> I had very similar problem with native app (unified-push-helloworld), >>> 400 code on first registration. >>> I'll check getToken() return value to verify that its actually same >>> issue. >>> >>> On Mon, Jun 20, 2016 at 10:28 AM, Julio Cesar Sanchez Hernandez < >>> jusanche(a)redhat.com&gt; wrote: >>> >>>> I think that should be enough, if the plugin uses the AGPush lib and >>>> the lib has that on the AndroidManifest, it should be merged with the app >>>> AndroidManifest. In the end a Cordova app is an Android app, it should work >>>> the same way the Android app works >>>> >>>> On Mon, Jun 20, 2016 at 10:02 AM, Matthias Wessendorf < >>>> matzew(a)apache.org&gt; wrote: >>>> >>>>> Ok, so IMO that sounds like a bug in the Android lib, so the UPS >>>>> registration should be fired after 'onTokenRefresh' is called >>>>> >>>>> Also, cordova question. Do we need to define our >>>>> "UnifiedPushInstanceIDListenerService" in the plugin's config section for >>>>> the manifest? Or is this, inside the AGPush lib good enough ? >>>>> >>>>> https://github.com/aerogear/aerogear-android-push/blob/master/aerogear-an... >>>>> >>>>> Thanks, >>>>> Matthias >>>>> >>>>> On Mon, Jun 20, 2016 at 9:46 AM, Julio Cesar Sanchez Hernandez < >>>>> jusanche(a)redhat.com&gt; wrote: >>>>> >>>>>> Yes, getToken will return null until onTokenRefresh is called the >>>>>> first time when it gets the real value. >>>>>> >>>>>> On Mon, Jun 20, 2016 at 9:06 AM, Matthias Wessendorf < >>>>>> matzew(a)apache.org&gt; wrote: >>>>>> >>>>>>> I looked here: >>>>>>> >>>>>>> https://developers.google.com/cloud-messaging/android/android-migrate-fcm... >>>>>>> >>>>>>> It says that we no longer need, but this: >>>>>>> >>>>>>> https://github.com/aerogear/aerogear-android-push/blob/3.0.1/aerogear-and... >>>>>>> >>>>>>> was replace with explicit FCM getToken() call: >>>>>>> >>>>>>> https://github.com/aerogear/aerogear-android-push/blob/4.0.0/aerogear-and... >>>>>>> >>>>>>> I think I am a bit over asked here, on the Android internals... >>>>>>> >>>>>>> Also, looks like we are not the only ones, seeing that problem :) >>>>>>> >>>>>>> http://stackoverflow.com/questions/37836383/android-firebase-token-is-nul... >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> On Sun, Jun 19, 2016 at 8:39 PM, Matthias Wessendorf < >>>>>>> matzew(a)apache.org&gt; wrote: >>>>>>> >>>>>>>> Looking at the JavaDoc for the getToken(): >>>>>>>> >>>>>>>> https://developers.google.com/android/reference/com/google/firebase/iid/F... >>>>>>>> >>>>>>>> "the master token or null if the token is not yet available" >>>>>>>> >>>>>>>> looks like on the android lib we should not have it return null ? >>>>>>>> >>>>>>>> On Sat, Jun 18, 2016 at 3:17 PM, Matthias Wessendorf < >>>>>>>> matzew(a)apache.org&gt; wrote: >>>>>>>> >>>>>>>>> Hi, >>>>>>>>> >>>>>>>>> when I launch a fresh and clean Cordova app (I am unable to >>>>>>>>> reproduce this on our native Android HelloWorld), and open it the first >>>>>>>>> time for registration, I am receiving a 400 status code from the server. >>>>>>>>> >>>>>>>>> Uninstalling the app, and launching it again, show the the >>>>>>>>> problem again, ONLY on Cordova... >>>>>>>>> >>>>>>>>> This means either the deviceToken from Android is null or invalid >>>>>>>>> (according to the pattern): >>>>>>>>> >>>>>>>>> https://github.com/aerogear/aerogear-unifiedpush-server/blob/1.1.x-dev/ja... >>>>>>>>> >>>>>>>>> Doing some debugging, on the server, I see the provided token is >>>>>>>>> actually null. >>>>>>>>> >>>>>>>>> Is there a race condition or so on the client, for the initial >>>>>>>>> launch of the app ? >>>>>>>>> >>>>>>>>> I added some logging on the client (see [1]), and indeed, it >>>>>>>>> looks like, on Cordova, it is actually null: >>>>>>>>> >>>>>>>>> https://gist.github.com/matzew/e66fa3e2bda6c4dd4655fbe5d63ed755#file-gist... >>>>>>>>> >>>>>>>>> Now... a few lines above... I see 'outdated' c2dm >>>>>>>>> packages/permissions, and a GCM: "Missmatched messenger", it looks like >>>>>>>>> something is still not correct. >>>>>>>>> >>>>>>>>> Any thoughts ? >>>>>>>>> >>>>>>>>> -Matthias >>>>>>>>> >>>>>>>>> [1] >>>>>>>>> https://github.com/matzew/aerogear-android-push/commit/1fa6aad53da4e1b9c4... >>>>>>>>> >>>>>>>>> -- >>>>>>>>> Matthias Wessendorf >>>>>>>>> >>>>>>>>> blog: http://matthiaswessendorf.wordpress.com/ >>>>>>>>> twitter: http://twitter.com/mwessendorf >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> Matthias Wessendorf >>>>>>>> >>>>>>>> blog: http://matthiaswessendorf.wordpress.com/ >>>>>>>> twitter: http://twitter.com/mwessendorf >>>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Matthias Wessendorf >>>>>>> >>>>>>> blog: http://matthiaswessendorf.wordpress.com/ >>>>>>> twitter: http://twitter.com/mwessendorf >>>>>>> >>>>>>> _______________________________________________ >>>>>>> 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 >>>>>> >>>>> >>>>> >>>>> >>>>> -- >>>>> Matthias Wessendorf >>>>> >>>>> blog: http://matthiaswessendorf.wordpress.com/ >>>>> twitter: http://twitter.com/mwessendorf >>>>> >>>>> _______________________________________________ >>>>> 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 >>>> >>> >>> >>> >>> -- >>> Oleg Matskiv >>> Associate Quality Engineer >>> Red Hat Mobile Application Platform >>> omatskiv(a)redhat.com >>> >> >> >> >> -- >> Oleg Matskiv >> Associate Quality Engineer >> Red Hat Mobile Application Platform >> omatskiv(a)redhat.com >> >> _______________________________________________ >> aerogear-dev mailing list >> aerogear-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/aerogear-dev >> > > > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > twitter: http://twitter.com/mwessendorf > -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ twitter: http://twitter.com/mwessendorf