crowetic/qortal
1
0
Fork 0
forked from Qortal/qortal
Code Pull Requests Activity

Return HTTP 400 code with useful message when a ParamException occurs from API call.

Browse Source
This commit is contained in:
catbref 2019-04-04 12:30:28 +01:00
parent 315ebff61d
commit 97142fdde8
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/org/qora/api/resource/ApiExceptionMapper.java
View File

@ -10,6 +10,7 @@ import javax.ws.rs.ext.Provider;
import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger; import org.apache.logging.log4j.Logger;
import org.glassfish.jersey.server.ParamException;
@Provider @Provider
public class ApiExceptionMapper implements ExceptionMapper<RuntimeException> { public class ApiExceptionMapper implements ExceptionMapper<RuntimeException> {
@ -26,6 +27,11 @@ public class ApiExceptionMapper implements ExceptionMapper<RuntimeException> {
public Response toResponse(RuntimeException e) { public Response toResponse(RuntimeException e) {
LOGGER.info(String.format("Exception %s during API call: %s", e.getClass().getCanonicalName(), request.getRequestURI())); LOGGER.info(String.format("Exception %s during API call: %s", e.getClass().getCanonicalName(), request.getRequestURI()));
if (e instanceof ParamException) {
ParamException pe = (ParamException) e;
return Response.status(Response.Status.BAD_REQUEST).entity("Bad parameter \"" + pe.getParameterName() + "\"").build();
}
if (e instanceof WebApplicationException) if (e instanceof WebApplicationException)
return ((WebApplicationException) e).getResponse(); return ((WebApplicationException) e).getResponse();