Re: [Pki-devel] [PATCH] 196 - Add decorator to handle exceptions
On 2/17/2014 11:38 AM, Ade Lee wrote:
Added decorator to handle exceptions
Decorator catches HttpErrorExceptions from Requests
and extracts the relevant PKIException object, and rethrows
it.
ACK. Just some comments:
1. In EXCEPTION_MAPPINGS is it possible to map to the exception class
directly? For example:
EXCEPTION_MAPPINGS = {
"com.netscape.certsrv.base.BadRequestException":
pki.BadRequestException,
...
}
This way it's not necessary to interpret the string class name.
2. I'm thinking we may not need the individual
<Resource>NotFoundException for Cert, Group, Profile, and User. So far I
don't see a lot of benefit in the Java version. We might be able to just
use the base ResourceNotFoundException.
--
Endi S. Dewata