[Pki-devel] [PATCH] 196 - Add decorator to handle exceptions
Endi Sukma Dewata
edewata at redhat.com
Tue Feb 18 19:55:45 UTC 2014
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
More information about the Pki-devel
mailing list