refactor(serverHandler): extract access log to log handler

This is to make sure log access request before processing
pre-middleware.

Author: marjune163

src/serverHandler/aliasHandler.go

@@ -135,8 +135,6 @@ func (h *aliasHandler) preCheck(w http.ResponseWriter, r *http.Request, data *re
 }
 
 func (h *aliasHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
-	h.logRequest(r)
-
 	// hsts redirect
 	if h.globalHsts && h.hsts(w, r) {
 		return

src/serverHandler/log.go

@@ -7,8 +7,8 @@ import (
 	"net/url"
 )
 
-func (h *aliasHandler) logRequest(r *http.Request) {
-	if !h.logger.CanLogAccess() {
+func logRequest(logger *serverLog.Logger, r *http.Request) {
+	if !logger.CanLogAccess() {
 		return
 	}
 
@@ -36,7 +36,7 @@ func (h *aliasHandler) logRequest(r *http.Request) {
 	}
 	buf = append(buf, uri...)
 
-	go h.logger.LogAccess(buf)
+	go logger.LogAccess(buf)
 }
 
 func (h *aliasHandler) logMutate(username, action, detail string, r *http.Request) {

src/serverHandler/logHandler.go

@@ -0,0 +1,23 @@
+package serverHandler
+
+import (
+	"mjpclab.dev/ghfs/src/serverLog"
+	"net/http"
+)
+
+type logHandler struct {
+	logger      *serverLog.Logger
+	nextHandler http.Handler
+}
+
+func (l logHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
+	logRequest(l.logger, r)
+	l.nextHandler.ServeHTTP(w, r)
+}
+
+func newLogHandler(logger *serverLog.Logger, nextHandler http.Handler) http.Handler {
+	return logHandler{
+		logger:      logger,
+		nextHandler: nextHandler,
+	}
+}

src/serverHandler/vhostHandler.go

@@ -104,7 +104,8 @@ func NewVhostHandler(
 	}
 
 	muxHandler := newMultiplexHandler(p, ap)
-	pathTransformHandler := newPathTransformHandler(p.PrefixUrls, muxHandler)
+	logHandler := newLogHandler(logger, muxHandler)
+	pathTransformHandler := newPathTransformHandler(p.PrefixUrls, logHandler)
 
 	return pathTransformHandler, nil
 }