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. 

Is there any downside to accepting results on the client side? (As opposed to configuring the library comply with a certain version of UPS)


-Matthias

--


_______________________________________________
aerogear-dev mailing list
aerogear-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-dev


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