feat: add remote logging to important errors

As convention always as 1st field, so it looks nice in console mode and matches existing logging

layer4/matchers.go

@@ -50,9 +50,10 @@ func (mset MatcherSet) Match(cx *Connection, logger *zap.Logger) (matched bool,
 		cx.rewind()
 		if logger.Core().Enabled(zap.DebugLevel) {
 			logger.Debug("matching",
+				zap.String("remote", cx.RemoteAddr().String()),
+				zap.Error(err),
 				zap.String("matcher", m.CaddyModule().String()),
 				zap.Bool("matched", matched),
-				zap.Error(err),
 			)
 		}
 		if !matched || err != nil {

layer4/routes.go

@@ -127,7 +127,7 @@ func wrapRoute(route *Route, logger *zap.Logger) Middleware {
 			// route must match at least one of the matcher sets
 			matched, err := route.matcherSets.AnyMatch(cx, logger)
 			if err != nil {
-				logger.Error("matching connection", zap.Error(err))
+				logger.Error("matching connection", zap.String("remote", cx.RemoteAddr().String()), zap.Error(err))
 				return nil // return nil so the error does not get logged again
 			}
 			if !matched {

layer4/server.go

@@ -106,7 +106,7 @@ func (s Server) handle(conn net.Conn) {
 	err := s.compiledRoute.Handle(cx)
 	duration := time.Since(start)
 	if err != nil {
-		s.logger.Error("handling connection", zap.Error(err))
+		s.logger.Error("handling connection", zap.String("remote", cx.RemoteAddr().String()), zap.Error(err))
 	}
 
 	s.logger.Debug("connection stats",

modules/l4tee/tee.go

@@ -103,7 +103,7 @@ func (t Handler) Handle(cx *layer4.Connection, next layer4.Handler) error {
 	go func() {
 		err := t.compiledChain.Handle(&branchc)
 		if err != nil {
-			t.logger.Error("handling connection in branch", zap.Error(err))
+			t.logger.Error("handling connection in branch", zap.String("remote", cx.RemoteAddr().String()), zap.Error(err))
 		}
 	}()