[aerogear-dev] Safari Push notifications

[Lucas Holmquist]

So i’ve run into a bit of a problem, I’m trying to send notifications,  using https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/push/src/main/java/org/jboss/aerogear/unifiedpush/message/sender/APNsPushNotificationSender.java#L52 <https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/push/src/main/java/org/jboss/aerogear/unifiedpush/message/sender/APNsPushNotificationSender.java#L52>

but i’m not sure how to also tell it to use the new SafarVariant.class that i’ve created. 

I’m not sure the syntax for https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/push/src/main/java/org/jboss/aerogear/unifiedpush/message/sender/APNsPushNotificationSender.java#L52 <https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/push/src/main/java/org/jboss/aerogear/unifiedpush/message/sender/APNsPushNotificationSender.java#L52>
to add a second thing in.

From the looks of it,  it looks like only one class is supported.

*Disclaimer:  javascript dude doing java*


> On Oct 24, 2014, at 10:24 AM, Matthias Wessendorf <matzew at apache.org> wrote:
> 
> 
> 
> On Friday, October 24, 2014, Lucas Holmquist <lholmqui at redhat.com <mailto:lholmqui at redhat.com>> wrote:
> 
>> On Oct 24, 2014, at 10:16 AM, Matthias Wessendorf <matzew at apache.org <>> wrote:
>> 
>> 
>> 
>> On Fri, Oct 24, 2014 at 3:36 PM, Lucas Holmquist <lholmqui at redhat.com <>> wrote:
>> Hello,
>> 
>> So i’ve decided to take today and try and add Safari Push notifications into the UPS.  When i tried about a year ago, i was successful, but i had lots of duplicated code since it is very close to the iOS variant.
>> basically, the only real difference is that Safari notifications use the Production cert only.  The other slight difference is how you send the message to the APNs service
>> 
>> I wondering if now is the time to start changing the variant names,  so iOSVariant would become APNsVariant(or something like that).
> 
> this would also apply to Android and Chrome since now both use the same GCM,
> 
> ah, right, almost forgot about that change :-)
>  
> 
>> 
>> +1
>>  
>> 
>> This would also change the endpoint at which we connect from ‘ios' to ‘apns’  .  Since this is sort of a breaking change, from a curl perspective,  does this need to wait until a Major release?
>> 
>> Let's have it for master (1.1.x series) only.
>> That said, since there may be a few mire 1.0.x releases, which also may have effect to the 1.1.x code base (merges), I'd like to wait a bit with the actual merge of "Safari push" to master.
>>  
>> 
>> plus this might make the current db migration stuff that is in progress more complex too
>> 
>> I have no problems keeping it separate, and then in a later release merging things together
>> 
>> ah, yeah - that's the best :) no need to "wait" on the work, let's keep branch in sync w/ master, and merge in a few weeks... ok ?
>>  
>> -Matthias
>> 
>> 
>> 
>> wdyt?
>> 
>> -Luke
>> 
>> 
>> 
>> 
>> 
>> 
>> _______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org <>
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev <https://lists.jboss.org/mailman/listinfo/aerogear-dev>
>> 
>> 
>> -- 
>> Matthias Wessendorf 
>> 
>> blog: http://matthiaswessendorf.wordpress.com/ <http://matthiaswessendorf.wordpress.com/>
>> sessions: http://www.slideshare.net/mwessendorf <http://www.slideshare.net/mwessendorf>
>> twitter: http://twitter.com/mwessendorf <http://twitter.com/mwessendorf>_______________________________________________
>> aerogear-dev mailing list
>> aerogear-dev at lists.jboss.org <>
>> https://lists.jboss.org/mailman/listinfo/aerogear-dev <https://lists.jboss.org/mailman/listinfo/aerogear-dev>
> 
> 
> -- 
> Sent from Gmail Mobile
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org <mailto:aerogear-dev at lists.jboss.org>
> https://lists.jboss.org/mailman/listinfo/aerogear-dev <https://lists.jboss.org/mailman/listinfo/aerogear-dev>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20141027/96ef5c16/attachment-0001.html