Added better logging for packets and payloads and now the writer is

responsible for packaging into a packet the payload (instead of everyone
who sends to the writer)

1 files changed, 12 insertions, 0 deletions

diff --git a/internal/packet/packet.go b/internal/packet/packet.go
index 8bc9669..a6a47e2 100644
--- a/internal/packet/packet.go
+++ b/internal/packet/packet.go
@@ -7,6 +7,7 @@ import (
 	"errors"
 	"fmt"
 	"io"
+	"log/slog"
 	"math"
 
 	"github.com/vmihailenco/msgpack/v5"
@@ -237,9 +238,20 @@ func (p Packet) Payload() []byte {
 }
 
 func (p Packet) String() string {
+	assert.Never("oops, found someone who uses packet.String(), improve this")
 	return fmt.Sprintf("Packet(v%v t%v %v [%v bytes...])", p.Version(), p.Encoding().String(), p.Type(), p.PayloadLength())
 }
 
+func (p Packet) LogValue() slog.Value {
+	return slog.GroupValue(
+		slog.Int("version", int(p.Version())),
+		slog.String("encoding", p.Encoding().String()),
+		slog.String("type", p.Type().String()),
+		slog.Int("payload_length", int(p.PayloadLength())),
+		slog.Int("total_bytes", len(p.data)),
+	)
+}
+
 func (p Packet) Into(writer io.Writer) (int, error) {
 	return writer.Write(p.data)
 }