> + case 402:
> + exception = new IllegalStateException(message, exception);
> + break;
> + case 404:
> + if
> (!command.getCurrentRequest().getMethod().equals("DELETE")) {
> + exception = new ResourceNotFoundException(message,
> exception);
> + }
> + break;
> + case 405:
> + exception = new IllegalArgumentException(message, exception);
> + break;
> + case 409:
> + exception = new IllegalStateException(message, exception);
> + break;
> + case 500:
> + exception = null;
I'm thinking it may be best to remove this entire ErrorHandler all together and
fall back to what I was doing it before, NullOn500, which is done on a
per-request basis. The 500 here is the sticky point and doing this for the
general case is not a good solution. The other solution is not great either but
it's better than this. I say we go with that, however unfortunate, until the
devs address the issue at some further date.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/116/files#r22196971
