perf: prevent large payload bytes print (#70)

yiweichi · web-flow · commit 54929786434f · 2025-11-28T22:19:48.000+08:00
diff --git a/encoding/codecv7.go b/encoding/codecv7.go
@@ -240,13 +240,13 @@ func (d *DACodecV7) checkCompressedDataCompatibility(payloadBytes []byte, checkL
 	}
 
 	if err = checkCompressedDataCompatibilityV7(compressedPayloadBytes); err != nil {
-		log.Warn("Compressed data compatibility check failed", "err", err, "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))
+		log.Warn("Compressed data compatibility check failed", "err", err, "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))
 		return nil, false, nil
 	}
 
 	// check if compressed data is bigger or equal to the original data -> no need to compress
 	if checkLength && len(compressedPayloadBytes) >= len(payloadBytes) {
-		log.Warn("Compressed data is bigger or equal to the original data", "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))
+		log.Warn("Compressed data is bigger or equal to the original data", "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))
 		return nil, false, nil
 	}
 
diff --git a/encoding/codecv8.go b/encoding/codecv8.go
@@ -52,13 +52,13 @@ func (d *DACodecV8) checkCompressedDataCompatibility(payloadBytes []byte, checkL
 	}
 
 	if err = checkCompressedDataCompatibilityV7(compressedPayloadBytes); err != nil {
-		log.Warn("Compressed data compatibility check failed", "err", err, "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))
+		log.Warn("Compressed data compatibility check failed", "err", err, "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))
 		return nil, false, nil
 	}
 
 	// check if compressed data is bigger or equal to the original data -> no need to compress
 	if checkLength && len(compressedPayloadBytes) >= len(payloadBytes) {
-		log.Warn("Compressed data is bigger or equal to the original data", "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))
+		log.Warn("Compressed data is bigger or equal to the original data", "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))
 		return nil, false, nil
 	}
 
diff --git a/encoding/codecv9.go b/encoding/codecv9.go
@@ -51,13 +51,13 @@ func (d *DACodecV9) checkCompressedDataCompatibility(payloadBytes []byte, checkL
 	}
 
 	if err = checkCompressedDataCompatibilityV9(compressedPayloadBytes); err != nil {
-		log.Warn("Compressed data compatibility check failed", "err", err, "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))
+		log.Warn("Compressed data compatibility check failed", "err", err, "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))
 		return nil, false, nil
 	}
 
 	// check if compressed data is bigger or equal to the original data -> no need to compress
 	if checkLength && len(compressedPayloadBytes) >= len(payloadBytes) {
-		log.Warn("Compressed data is bigger or equal to the original data", "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))
+		log.Warn("Compressed data is bigger or equal to the original data", "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))
 		return nil, false, nil
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -240,13 +240,13 @@ func (d *DACodecV7) checkCompressedDataCompatibility(payloadBytes []byte, checkL`
`240`	`240`	`}`
`241`	`241`
`242`	`242`	`if err = checkCompressedDataCompatibilityV7(compressedPayloadBytes); err != nil {`
`243`		`- log.Warn("Compressed data compatibility check failed", "err", err, "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))`
	`243`	`+ log.Warn("Compressed data compatibility check failed", "err", err, "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))`
`244`	`244`	`return nil, false, nil`
`245`	`245`	`}`
`246`	`246`
`247`	`247`	`// check if compressed data is bigger or equal to the original data -> no need to compress`
`248`	`248`	`if checkLength && len(compressedPayloadBytes) >= len(payloadBytes) {`
`249`		`- log.Warn("Compressed data is bigger or equal to the original data", "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))`
	`249`	`+ log.Warn("Compressed data is bigger or equal to the original data", "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))`
`250`	`250`	`return nil, false, nil`
`251`	`251`	`}`
`252`	`252`
Original file line number	Diff line number	Diff line change
`@@ -52,13 +52,13 @@ func (d *DACodecV8) checkCompressedDataCompatibility(payloadBytes []byte, checkL`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`if err = checkCompressedDataCompatibilityV7(compressedPayloadBytes); err != nil {`
`55`		`- log.Warn("Compressed data compatibility check failed", "err", err, "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))`
	`55`	`+ log.Warn("Compressed data compatibility check failed", "err", err, "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))`
`56`	`56`	`return nil, false, nil`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`// check if compressed data is bigger or equal to the original data -> no need to compress`
`60`	`60`	`if checkLength && len(compressedPayloadBytes) >= len(payloadBytes) {`
`61`		`- log.Warn("Compressed data is bigger or equal to the original data", "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))`
	`61`	`+ log.Warn("Compressed data is bigger or equal to the original data", "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))`
`62`	`62`	`return nil, false, nil`
`63`	`63`	`}`
`64`	`64`
Original file line number	Diff line number	Diff line change
`@@ -51,13 +51,13 @@ func (d *DACodecV9) checkCompressedDataCompatibility(payloadBytes []byte, checkL`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`if err = checkCompressedDataCompatibilityV9(compressedPayloadBytes); err != nil {`
`54`		`- log.Warn("Compressed data compatibility check failed", "err", err, "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))`
	`54`	`+ log.Warn("Compressed data compatibility check failed", "err", err, "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))`
`55`	`55`	`return nil, false, nil`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`// check if compressed data is bigger or equal to the original data -> no need to compress`
`59`	`59`	`if checkLength && len(compressedPayloadBytes) >= len(payloadBytes) {`
`60`		`- log.Warn("Compressed data is bigger or equal to the original data", "payloadBytes", hex.EncodeToString(payloadBytes), "compressedPayloadBytes", hex.EncodeToString(compressedPayloadBytes))`
	`60`	`+ log.Warn("Compressed data is bigger or equal to the original data", "payloadLen", len(payloadBytes), "compressedPayloadLen", len(compressedPayloadBytes))`
`61`	`61`	`return nil, false, nil`
`62`	`62`	`}`
`63`	`63`