From 515aa7108bee77fa96278056b2cc1fff77a7ee83 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Josep=20M=C2=AA=20Domingo?= <jdomingocatafal@gmail.com>
Date: Thu, 25 Mar 2021 04:17:36 +0100
Subject: [PATCH] Move logger middleware to capture routing errors (ex: 405).
(#877)
* Fix #836
* Remove requestLogger middleware from MountRouter
---
server/server.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/server/server.go b/server/server.go
index 1948c5cb4..5a03086b9 100644
--- a/server/server.go
+++ b/server/server.go
@@ -38,7 +38,6 @@ func (a *Server) MountRouter(urlPath string, subRouter Handler) {
log.Info("Mounting routes", "path", urlPath)
subRouter.Setup(urlPath)
a.router.Group(func(r chi.Router) {
- r.Use(requestLogger)
r.Mount(urlPath, subRouter)
})
}
@@ -59,6 +58,7 @@ func (a *Server) initRoutes() {
r.Use(middleware.Compress(5, "application/xml", "application/json", "application/javascript"))
r.Use(middleware.Heartbeat("/ping"))
r.Use(injectLogger)
+ r.Use(requestLogger)
r.Use(robotsTXT(ui.Assets()))
indexHtml := path.Join(conf.Server.BaseURL, consts.URLPathUI, "index.html")