feat: change skip message log level to debug and add Pretty for writi…

…ng headers (#137)

* feat: change skip message log level to debug and add Pretty for writing headers

consumer_base.go

@@ -205,19 +205,19 @@ func (c *base) startConsume() {
 				continue
 			}
 
+			incomingMessage := &IncomingMessage{
+				kafkaMessage: m,
+				message:      fromKafkaMessage(m),
+			}
+
 			if c.skipMessageByHeaderFn != nil && c.skipMessageByHeaderFn(m.Headers) {
-				c.logger.Infof("Message is not processed. Header filter applied. Headers: %v", m.Headers)
+				c.logger.Debugf("Message is not processed. Header filter applied. Headers: %v", incomingMessage.message.Headers.Pretty())
 				if err = c.r.CommitMessages([]kafka.Message{*m}); err != nil {
 					c.logger.Errorf("Commit Error %s,", err.Error())
 				}
 				continue
 			}
 
-			incomingMessage := &IncomingMessage{
-				kafkaMessage: m,
-				message:      fromKafkaMessage(m),
-			}
-
 			if c.distributedTracingEnabled {
 				incomingMessage.message.Context = c.propagator.Extract(context.Background(), otelkafkakonsumer.NewMessageCarrier(m))
 			}

message.go

@@ -2,6 +2,8 @@ package kafka
 
 import (
 	"context"
+	"fmt"
+	"strings"
 	"time"
 
 	kcronsumer "github.com/Trendyol/kafka-cronsumer/pkg/kafka"
@@ -15,6 +17,17 @@ const (
 
 type Header = protocol.Header
 
+type Headers []Header
+
+// Pretty  Writes every header key and value, it is useful for debugging purpose
+func (hs Headers) Pretty() string {
+	headerStrings := make([]string, len(hs))
+	for i := range hs {
+		headerStrings[i] = fmt.Sprintf("%s: %s", hs[i].Key, string(hs[i].Value))
+	}
+	return strings.Join(headerStrings, ", ")
+}
+
 type Message struct {
 	Time       time.Time
 	WriterData interface{}
@@ -24,7 +37,7 @@ type Message struct {
 	Topic         string
 	Key           []byte
 	Value         []byte
-	Headers       []Header
+	Headers       Headers
 	Partition     int
 	Offset        int64
 	HighWaterMark int64

message_test.go

@@ -266,3 +266,19 @@ func TestMessage_toRetryableMessage(t *testing.T) {
 		}
 	})
 }
+
+func TestHeaders_Pretty(t *testing.T) {
+	// Given
+	headers := Headers{
+		{Key: "key1", Value: []byte("value1")},
+		{Key: "key2", Value: []byte("value2")},
+	}
+
+	// When
+	result := headers.Pretty()
+
+	// Then
+	if result != "key1: value1, key2: value2" {
+		t.Error("result must be `key1: value1, key2: value2`")
+	}
+}