chore: provide more logging for dns requests

Annotate dns request debug log with additional fields like `proto` and `addr`.

For #10308

Signed-off-by: Dmitriy Matrenichev <[email protected]>
(cherry picked from commit bf0f910)

Author: DmitriyMV

internal/pkg/dns/dns.go

@@ -118,8 +118,13 @@ func (h *Handler) ServeDNS(ctx context.Context, wrt dns.ResponseWriter, msg *dns
 	defer h.mx.RUnlock()
 
 	req := request.Request{W: wrt, Req: msg}
-
-	h.logger.Debug("dns request", zap.Stringer("data", msg))
+	logger := h.logger.With(
+		zap.Stringer("data", msg),
+		zap.String("proto", req.Proto()),
+		zap.String("question", req.QName()),
+		zap.Stringer("local_addr", wrt.LocalAddr()),
+		zap.Stringer("remote_addr", wrt.RemoteAddr()),
+	)
 
 	var (
 		called bool
@@ -131,6 +136,8 @@ func (h *Handler) ServeDNS(ctx context.Context, wrt dns.ResponseWriter, msg *dns
 		called = true
 		opts := proxy.Options{}
 
+		logger.Debug("making dns request", zap.String("upstream", ups.Addr()))
+
 		for {
 			resp, err = ups.Connect(ctx, req, opts)
 
@@ -168,18 +175,18 @@ func (h *Handler) ServeDNS(ctx context.Context, wrt dns.ResponseWriter, msg *dns
 	}
 
 	if !req.Match(resp) {
-		h.logger.Warn("dns response didn't match", zap.Stringer("data", resp))
+		logger.Warn("dns response didn't match", zap.Stringer("data", resp))
 
 		return dns.RcodeFormatError, nil
 	}
 
 	err = wrt.WriteMsg(resp)
 	if err != nil {
 		// We can't do much here, but at least log the error.
-		h.logger.Warn("error writing dns response", zap.Error(err))
+		logger.Warn("error writing dns response", zap.Error(err))
 	}
 
-	h.logger.Debug("dns response", zap.Stringer("data", resp))
+	logger.Debug("dns response", zap.Stringer("data", resp))
 
 	return dns.RcodeSuccess, nil
 }