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.