[aerogear-dev] [Android Push SDK] ctor of Registrar
Summers Pittman
supittma at redhat.com
Mon Jul 1 13:35:17 EDT 2013
On 07/01/2013 03:30 AM, Matthias Wessendorf wrote:
> any comments ?
Yeah it is probably a good idea. :)
PushConfig could have endpoints added to it and default to
/rest/resgistry/device.
>
>
> On Tue, Jun 25, 2013 at 6:37 PM, Matthias Wessendorf
> <matzew at apache.org <mailto:matzew at apache.org>> wrote:
>
> Hi,
>
> I noticed that the ctor, currently, takes the entire URL of the
> "registration" endpoint:
> https://github.com/secondsun/PushEEDemo/blob/master/src/com/example/pusheedemo/MainApplication.java#L23
>
> That's ok, for now. but I think that ideally the application
> specific part (here "/rest/registry/device") should not be
> exposed. It could be changed, by PushEE.
>
> In the iOS SDK, we only "require" the URL + the name of the
> servlet context.
>
> For instance:
> https://github.com/aerogear/aerogear-push-ios-registration/blob/master/push-sdkTests/AGDeviceRegistrationSpec.m#L47
> (from a "mocked" test)
>
> So the SDK internally appends the "rest/registry/device":
> https://github.com/aerogear/aerogear-push-ios-registration/blob/master/push-sdk/AGDeviceRegistration.m#L71
>
> Would you think this is good for the Android-SDK as well ?
>
>
> Greetings.
> Matthias
>
>
> --
> Matthias Wessendorf
>
> blog: http://matthiaswessendorf.wordpress.com/
> sessions: http://www.slideshare.net/mwessendorf
> twitter: http://twitter.com/mwessendorf
>
>
>
>
> --
> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130701/0d1c0e07/attachment.html
More information about the aerogear-dev
mailing list