Merge pull request #678 from code-payments/fix/grpc-logging-large-bytestring

bmc08gt · web-flow · commit e09eca5da594 · 2026-05-04T16:36:48.000-04:00
fix(oc): replace hex dump of ByteString fields with size summary in gRPC logging
diff --git a/services/opencode/src/main/kotlin/com/getcode/opencode/utils/logging/LoggingClientCallListener.kt b/services/opencode/src/main/kotlin/com/getcode/opencode/utils/logging/LoggingClientCallListener.kt
@@ -135,7 +135,7 @@ private fun formatFieldValue(fieldName: String, value: Any): String? = when (val
         if (value.serializedSize == 0) null else formatProto(value)
     }
     is ByteString -> {
-        if (value.isEmpty) null else value.toByteArray().toHex().masked()
+        if (value.isEmpty) null else "<${value.size()} bytes>"
     }
     is List<*> -> {
         if (value.isEmpty()) null

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ private fun formatFieldValue(fieldName: String, value: Any): String? = when (val`
`135`	`135`	`if (value.serializedSize == 0) null else formatProto(value)`
`136`	`136`	`}`
`137`	`137`	`is ByteString -> {`
`138`		`- if (value.isEmpty) null else value.toByteArray().toHex().masked()`
	`138`	`+ if (value.isEmpty) null else "<${value.size()} bytes>"`
`139`	`139`	`}`
`140`	`140`	`is List<*> -> {`
`141`	`141`	`if (value.isEmpty()) null`