[aerogear-dev] USP 1.1.0 - API change: Status code change for Sender Endpoint

[Summers Pittman]

On 02/04/2015 10:04 AM, Matthias Wessendorf wrote:
>
>
> On Wed, Feb 4, 2015 at 3:56 PM, Summers Pittman <supittma at redhat.com 
> <mailto:supittma at redhat.com>> wrote:
>
>     On 02/03/2015 04:13 PM, Matthias Wessendorf wrote:
>>     Hi,
>>
>>     I recently noticed that we are (IMO) incorrectly returning 200 (OK):
>>     https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/jaxrs/src/main/java/org/jboss/aerogear/unifiedpush/rest/sender/PushNotificationSenderEndpoint.java#L112-L113
>>
>>     Since we really do submit an async job here, and its processing
>>     may not have been started when we do return the 200 (OK), I'd
>>     like to propose a slight change here to instead use 202 (Accepted):
>>
>>     http://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html#sec10.2.3
>>
>>     If the team is OK (ha ha) with this, I will track this in JIRA
>     This should be OK, I will need to review the Android code.
>
>
> I hope no Android code is actually using the SENDER api :-)
Sorry pre coffee reading comprehension. I thought you were talking about 
for registration flavored things for some reason.
>
>
>     Is there any downside to accepting results on the client side? (As
>     opposed to configuring the library comply with a certain version
>     of UPS)
>
>
> Not sure what you are asking here
>
>
>>
>>     -Matthias
>>
>>     -- 
>>     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  <mailto:aerogear-dev at lists.jboss.org>
>>     https://lists.jboss.org/mailman/listinfo/aerogear-dev
>
>
>     -- 
>     Summers Pittman
>     >>Phone:404 941 4698
>     >>Java is my crack.
>
>
>     _______________________________________________
>     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
>
>
>
>
> -- 
> 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


-- 
Summers Pittman
>>Phone:404 941 4698
>>Java is my crack.

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