Hi all,

It seems $subject is possible. We wrap real exceptions
with UserManagerException in this class. IMO this is not correct way of
handling exceptions.

As a result of this, it returns wrong response.

E.g. Removing non existing user returns "Could not delete user: sd", but it
should return that user does not exist.

stratos>  remove-user sd
Could not delete user: sd

Correct way is to let the "StratosApiV41" class handle all the specific
exceptions and return correct status code with message.

Wdyt?

Thanks.
-- 
--
Lahiru Sandaruwan
Committer and PMC member, Apache Stratos,
Senior Software Engineer,
WSO2 Inc., http://wso2.com
lean.enterprise.middleware

phone: +94773325954
email: lahi...@wso2.com blog: http://lahiruwrites.blogspot.com/
linked-in: http://lk.linkedin.com/pub/lahiru-sandaruwan/16/153/146

Reply via email to