[aerogear-dev] Validation for creating/updating push apps and their variants

[Matthias Wessendorf]

Following up on [1], currently we give 400 (Bad Request), with no good
meaning, when creating/updating a PushApplication or a Variant;


The HTML page(!) simply says "The request sent by the client was
syntactically incorrect "

As suggested in [1] a good idea is using Bean Validation and returning
better errors.

The result would be a 400 (Bad Request) response, containing the errors and
their messages as a JSON object (instead of a HTML page).

E.g.:

{"name":"may not be null"}

This is a) more meaningful and we can also get rid of our 'hard-coded'
validation rules, like:
https://github.com/aerogear/aerogear-unifiedpush-server/blob/master/src/main/java/org/jboss/aerogear/unifiedpush/rest/registry/applications/AndroidVariantEndpoint.java#L79-L82

-Matthias


[1] https://issues.jboss.org/browse/AGPUSH-255

-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/aerogear-dev/attachments/20130918/20714e36/attachment.html