<div dir="ltr">+1.<div><br></div><div>This is something we will need to integrate UPS with RHMAP. At the moment, when user updates the network details for android push, we don't get the updated data back from UPS, which means we can't update the the apps with the correct information. We can issue another GET request for it, but it is a PITA and ideally we would expect the PUT endpoint should return the full data rather than just a status code.</div><div><br></div><div>Thanks.</div><div><br></div><div>Regards,</div><div>Wei Li</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 4, 2015 at 2:33 PM, Matthias Wessendorf <span dir="ltr"><<a href="mailto:matzew@apache.org" target="_blank">matzew@apache.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi,<br></div><div><br></div><div>currently when updating the network details, we do a PUT and srver returns 204 - not content, see:</div><div><a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/jaxrs/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/AndroidVariantEndpoint.java#L112-L142" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/jaxrs/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/AndroidVariantEndpoint.java#L112-L142</a></div><div><br></div><div><br></div><div>I wonder if we should actually return the updated values as part of the the response - this would mean we will be consistent to the endpoint to update the secret. There we do a PUT as well, but reasonably return the updated entity:</div><div><a href="https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/jaxrs/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/AbstractVariantEndpoint.java#L61-L78" target="_blank">https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/jaxrs/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/AbstractVariantEndpoint.java#L61-L78</a></div><div><br></div><div>This means, it's also easier to actually integrate the entire REST APIs into other, larger systems.</div><div><br></div><div>Any thoughts?</div><span class="HOEnZb"><font color="#888888"><div>-Matthias</div><div><br></div>-- <br><div>Matthias Wessendorf <br><br>blog: <a href="http://matthiaswessendorf.wordpress.com/" target="_blank">http://matthiaswessendorf.wordpress.com/</a><br>sessions: <a href="http://www.slideshare.net/mwessendorf" target="_blank">http://www.slideshare.net/mwessendorf</a><br>twitter: <a href="http://twitter.com/mwessendorf" target="_blank">http://twitter.com/mwessendorf</a></div>
</font></span></div>
<br>_______________________________________________<br>
aerogear-dev mailing list<br>
<a href="mailto:aerogear-dev@lists.jboss.org">aerogear-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Wei Li<br>Red Hat email address: <a href="mailto:wei.li@redhat.com" target="_blank">wei.li@redhat.com</a><br>Senior Software Engineer</div></div>
</div>