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

Sébastien Blanc scm.blanc at gmail.com
Tue Feb 3 16:57:37 EST 2015


Agreed but we should make that clear in the clients (Java and Node) when receiving the response (I'm thinking about the callbacks) , that said, an javadoc/comment should de enough.

Envoyé de mon iPhone

> Le 3 févr. 2015 à 22:13, Matthias Wessendorf <matzew at apache.org> a écrit :
> 
> 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
> 
> -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
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20150203/5d49b37a/attachment.html 


More information about the aerogear-dev mailing list