Print in a place that hopefully survives more errors

cmd/http-log/main.go

@@ -271,7 +271,6 @@ func main() {
 	}
 
 	loggingMux := handlers.NewLogMiddle(
-		op,
 		actionMux,
 	)
 
@@ -337,11 +336,16 @@ func main() {
 		// Called when an http server connection changes state
 		// Called by h2.
 		ConnState: func(c net.Conn, cs http.ConnState) {
-			_, log := fromHackStore(c, log)
+			ctx, log := fromHackStore(c, log)
 
 			log.Info("Connection state change", "state", cs)
 
 			if cs == http.StateClosed {
+				srvData := ctxt.SrvDataFromContext(ctx)
+				reqData := ctxt.ReqDataFromContext(ctx)
+				respData := ctxt.RespDataFromContext(ctx)
+				op.Output(srvData, reqData, respData)
+
 				delete(hackStore, c.RemoteAddr())
 				log.Debug("Connection closed; removing entry from hackStore", "remoteAddr", c.RemoteAddr())
 			}

melange.yaml

@@ -1,7 +1,7 @@
 package:
   name: http-log
   epoch: 0
-  version: v0.7.17-8-gb993e41-dirty
+  version: v0.7.17-9-ga361687-dirty
   description: Logs HTTP requests
   url: https://github.com/mt-inside/http-log
   copyright:

pkg/handlers/log_middle.go

@@ -10,21 +10,17 @@ import (
 	"github.com/mt-inside/http-log/internal/ctxt"
 	"github.com/mt-inside/http-log/pkg/enricher"
 	"github.com/mt-inside/http-log/pkg/extractor"
-	"github.com/mt-inside/http-log/pkg/output"
 	"github.com/mt-inside/http-log/pkg/parser"
 )
 
 type LogMiddle struct {
-	op   output.Renderer
 	next http.Handler
 }
 
 func NewLogMiddle(
-	op output.Renderer,
 	next http.Handler,
 ) http.Handler {
 	return &LogMiddle{
-		op,
 		next,
 	}
 }
@@ -38,7 +34,6 @@ func (lm LogMiddle) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 
 	srvData := ctxt.SrvDataFromContext(ctx)
 	reqData := ctxt.ReqDataFromContext(ctx)
-	respData := ctxt.RespDataFromContext(ctx)
 
 	/* Record request info */
 
@@ -63,7 +58,7 @@ func (lm LogMiddle) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 
 	/* Print */
 
-	lm.op.Output(srvData, reqData, respData)
+	// Printing is done when ConnState(closed)
 
 	/* Done */