Log the request path of a failed request in the ErrorMiddleware.

[maciejtrybilo]

Added the request path to the logging from the ErrorMiddleware. For instance when a resource /hello/vapor cannot be found currently the log will read Abort.404: Not Found. After this change the log will read Abort.404: /hello/vapor Not Found which will improve monitoring.

[0xTim]

@maciejtrybilo could you add a test to show the path being logged?

The only difference I can see is that this logs the path of every failed request, not just a 404. But I actually quite like that!

[maciejtrybilo]

@0xTim Yep, that's why I've opened a separate PR as the functionality is a little different.

Added some tests. Let me know what you think!

[tanner0101]

+1 this approach is much better, thank you.

[maciejtrybilo]

Thanks for the comments @tanner0101 Let me know anything else.

[tanner0101]

This is great, thanks @maciejtrybilo !

[tanner0101]

Released in https://github.com/vapor/vapor/releases/tag/3.3.3

[natanrolnik]

@maciejtrybilo thanks for the PR! From what I looked now, it seems that in ErrorMiddleware in the Vapor 4 Beta we don't have this behavior yet, right @tanner0101 ?
Could this be achieved with a similar approach?

[tanner0101]

@natanrolnik yeah that's true. An issue for Vapor 4 specifically would be appreciated.