[aerogear-dev] Differences between Firefox OS "native" Push lib and AeroGear's Push adapter

Lucas Holmquist lholmqui at redhat.com
Wed Jul 23 15:59:33 EDT 2014


would help if i actually put the issue in https://issues.jboss.org/browse/AGPUSH-833
On Jul 23, 2014, at 3:56 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:

> 
> so there is this issue,  , about not being able to delete a simplePush installation.
> 
> using the UPS js client,  i registered the “deviceToken” with the push server, but when trying to use CURL or even the JS lib,  i was getting a 404, and i noticed that it would never actually hit the server( i was in debug mode in IntelliJ )
> 
> this was a demo that i had working back in april https://github.com/lholmquist/mobileweek/blob/master/mobileweek-ffos/js/app.js#L17
> 
> i noticed that when i registered the deviceToken, i was doing a encodeURIComponent before sending it to the UPS.
> 
> i modified the UPS JS client to do it this way, and then when i tried to do the remove,  i actually hit the breakpoints i set.  but the server could not find the installation doing 
> 
> Installation installation =
>                 clientInstallationService.findInstallationForVariantByDeviceToken(variant.getVariantID(), token);
> 
> 
> I changed the JS client to only do 1 encode instead of 2 but it still couldn’t find the installation.
> 
> I’m not really familiar with JPA and all that,  so i don’t know how the query stuff works in this case
> 
> 
> But i do think we should be encoding the deviceToken( which is the pushEndpointURL ) when registering with the UPS, or else i don’t think we can ever get access to that endpoint
> 
> 
> On Jul 14, 2014, at 9:52 AM, Lucas Holmquist <lholmqui at redhat.com> wrote:
> 
>> 
>> On Jul 11, 2014, at 6:33 AM, Matthias Wessendorf <matzew at apache.org> wrote:
>> 
>>> Hey Luke,
>>> 
>>> I created this JIRA:
>>> https://issues.jboss.org/browse/AGPUSH-802
>>> 
>>> As I plan working on AGPUSH-532 next week, to make sure we have the 'right' model in place for 1.0.0. I think once the server bits are merged, we should address AGPUSH-802 
>> 
>> kk
>> 
>>> 
>>> -Matthias
>>> 
>>> 
>>> On Thu, Apr 10, 2014 at 5:22 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:
>>> so after testing this thing out with encodeURIComponent,  the delete does work,  however,  we need to "double up" the encode.
>>> 
>>> this works:
>>> 
>>> https://gist.github.com/lholmquist/10393357
>>> 
>>> 
>>> this doesn't:
>>> 
>>> https://gist.github.com/lholmquist/10393450
>>> 
>>> 
>>> 
>>> On Apr 7, 2014, at 2:09 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:
>>> 
>>>> 
>>>> On Apr 7, 2014, at 12:52 PM, Matthias Wessendorf <matzew at apache.org> wrote:
>>>> 
>>>>> Ok,
>>>>> 
>>>>> So as token, we use the endpointURL, and on client we perform  the encodeURIComponent() function ?
>>>> 
>>>> i think so( as suggested 2 months ago  :) )   i just need to see how this will affect how we store stuff in LS.
>>>> 
>>>> 
>>>> 
>>>>> 
>>>>> -Matthias
>>>>> 
>>>>> On Monday, April 7, 2014, Lucas Holmquist <lholmqui at redhat.com> wrote:
>>>>> so i went back to look at what i had,
>>>>> 
>>>>> 
>>>>> i don't think we need to get to complicated here,
>>>>> 
>>>>> reading the spec stuff, and this example
>>>>> 
>>>>> https://wiki.mozilla.org/WebAPI/SimplePush#Client_.28WebApp.29_code
>>>>> 
>>>>> they show sending the pushEndpoint to the "App server",  so i think we could just use and keep it simple
>>>>> 
>>>>> it is also recommended that the channelID is never exposed to the application.
>>>>> 
>>>>> 
>>>>> On Apr 1, 2014, at 3:34 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:
>>>>> 
>>>>>> i had something,  now i forgot what it was,   need to go back and check
>>>>>> On Apr 1, 2014, at 3:05 PM, Matthias Wessendorf <matzew at apache.org> wrote:
>>>>>> 
>>>>>>> 
>>>>>>> On Fri, Feb 14, 2014 at 2:06 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:
>>>>>>> still exploring
>>>>>>> 
>>>>>>> :-) any recent thoughts on 'encodeURIComponent()' ? 
>>>>>>>  
>>>>>>> 
>>>>>>> On Feb 13, 2014, at 3:39 PM, Sebastien Blanc <scm.blanc at gmail.com> wrote:
>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>>> 
>>>>>>>> On Wed, Feb 12, 2014 at 2:30 PM, Lucas Holmquist <lholmqui at redhat.com> wrote:
>>>>>>>> i might have a couple thoughts, but i need to try some things out first
>>>>>>>> 
>>>>>>>> Any update on that or does the solution proposed by Matzew (using  encodeURIComponent() ) could be enough ? 
>>>>>>>> 
>>>>>>>> On Feb 12, 2014, at 3:53 AM, Sebastien Blanc <scm.blanc at gmail.com> wrote:
>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> On Tue, Feb 11, 2014 at 7:15 PM, Sebastien Blanc <scm.blanc at gmail.com> wrote:
>>>>>>>>> Ok,
>>>>>>>>> I've been doing some tests by using the PushEndpoint as device token. For registration it works but I just faced an issue by trying to unregister because the URL for the DELETE looks like : 
>>>>>>>>> 
>>>>>>>>> https://judconpush-sblanc.rhcloud.com/rest/registry/device/https://updates.push.services.mozilla.com/update/my_personnal_psuhendpoint_id [
>>>>>>>>> 
>>>>>>>>> And the REST endpoint get a bit crazy by the extra "/" present in the endpoint URL. Therefore, I think we must just use the last URL fragment as deviceToken.
>>>>>>>>>  
>>>>>>>>> Ok answering to myself ;) That won't work neither since if we do that UPS won't have the compllete push endpoint URL. 
>>>>>>>>> So how do we deal with that ?
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> On Thu, Feb 6, 2014 at 3:27 PM, Matthias Wessendorf <matzew at apache.org> wrote:
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> 
>>>>>>>>> On Thu, Feb 6, 2014 at 3:11 PM, Sebastien Blanc
>>>>> 
>>>>> 
>>>>> -- 
>>>>> Sent from Gmail Mobile
>>>>> _______________________________________________
>>>>> aerogear-dev mailing list
>>>>> aerogear-dev at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>>> 
>>>> _______________________________________________
>>>> aerogear-dev mailing list
>>>> aerogear-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>> 
>>> 
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>>> 
>>> 
>>> 
>>> -- 
>>> Matthias Wessendorf 
>>> 
>>> blog: http://matthiaswessendorf.wordpress.com/
>>> sessions: http://www.slideshare.net/mwessendorf
>>> twitter: http://twitter.com/mwessendorf
>>> _______________________________________________
>>> aerogear-dev mailing list
>>> aerogear-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
>> 
>> _______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20140723/f4cce786/attachment.html 


More information about the aerogear-dev mailing list