在 DropWizard 或 Jetty 应用程序中记录导致 404 的 URL

Question

我们是运行 DropWizard 并试图删除导致抛出 404 响应的 URLs 的日志记录

我们有一个接收 NotFoundException 的包罗万象的异常映射器。令人沮丧的是，该异常没有携带 URL 导致它被抛出的上下文。

此处的示例应用程序：https://github.com/pauldambra/not.found.example

我们正在使用 ExceptionMapper

public class NotFoundLogger implements ExceptionMapper<NotFoundException> {

    ExampleLogger logger = new ExampleLogger();

    @Override
    public Response toResponse(final NotFoundException exception) {
        logger.error(urlFrom(exception), exception);
        return Response.status(404).build();
    }

    private String urlFrom(final NotFoundException exception) {
        return "why is this not a property on the exception?!";
    }

    private class ExampleLogger {
        void error(final String notFoundUrl, final NotFoundException exception) {
            System.out.println("someone tried to load " + notFoundUrl);
            System.out.println(exception.getMessage());
        }
    }
}

如果我们在有人请求 URL 应用程序不提供服务时查看应用程序日志，我们会看到应用程序可以记录它正在为路径返回 404，但我们的自定义记录器没有访问 URL

someone tried to load why is this not a property on the exception?!
HTTP 404 Not Found
127.0.0.1 - - [08/May/2019:09:53:47 +0000] "GET /ping/pong HTTP/1.1" 404

这样做 ExceptionMapper 是错误的方法吗？

Answer 1

原来有两种方法

有人确实使用异常映射器：

public class NotFoundLogger implements ExceptionMapper<NotFoundException> {

    // magically inject a thing
    // remember that magic is for evil wizards
    @Context
    private HttpServletRequest request;

    private ExampleLogger logger = new ExampleLogger();

    @Override
    public Response toResponse(final NotFoundException exception) {
        final StringBuffer absolutePath = HttpUtils.getRequestURL(request);
        logger.error("exception mapper: " + absolutePath, exception);
        return Response.status(404).build();
    }
}

这可行，但不是很容易被发现。

您还可以添加响应过滤器

public class NotFoundLoggingFilter implements ContainerResponseFilter {
    private ExampleLogger logger = new ExampleLogger();

    @Override
    public void filter(ContainerRequestContext requestContext,
                       ContainerResponseContext responseContext) {
        if (responseContext.getStatus() != 404) {
            return;
        }

        final URI absolutePath = requestContext.getUriInfo().getAbsolutePath();
        logger.error("filter: " + absolutePath, new NotFoundException());
    }
}

这不需要任何魔法，所以很适合我，但你可以选择你的毒药。

确实路径应该在 NotFoundException - 如果我有更多时间，我会建议更改代码以添加它。

在 DropWizard 或 Jetty 应用程序中记录导致 404 的 URL

Log URL that caused 404 in a DropWizard or Jetty application

java

jetty

jax-rs

dropwizard