https://github.com/aerogear/aerogear-unifiedpush-server/pull/726

On Thu, May 19, 2016 at 4:10 PM, Luke Holmquist <lholmqui@redhat.com> wrote:


On Thu, May 19, 2016 at 4:09 PM, Matthias Wessendorf <matzew@apache.org> wrote:

i thought it sounded familiar


Systems that integrate with the UPS might appreciate the actual response data of the updated entity, so we moved to 200 status code.

The doc is broken. Good find Luke. 

Mind sending a doc patch ? 

Doc Patch incoming

On Thu, May 19, 2016 at 9:21 PM, Luke Holmquist <lholmqui@redhat.com> wrote:
The variant update endpoint returns a 200 with the updated content, but the docs says it should return a 204 with no Content

i've create a JIRA for it,  https://issues.jboss.org/browse/AGPUSH-1635


I think PUT's are suppose to be 204's, but there might be things relying on the fact that the update returns data,

here is a link to the AndroidVariants update endpoint,  https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/jaxrs/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/AndroidVariantEndpoint.java#L142

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



--

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