Skip to content

fix(oc): replace hex dump of ByteString fields with size summary in gRPC logging#678

Merged
bmc08gt merged 1 commit intocode/cashfrom
fix/grpc-logging-large-bytestring
May 4, 2026
Merged

fix(oc): replace hex dump of ByteString fields with size summary in gRPC logging#678
bmc08gt merged 1 commit intocode/cashfrom
fix/grpc-logging-large-bytestring

Conversation

@bmc08gt
Copy link
Copy Markdown
Collaborator

@bmc08gt bmc08gt commented May 4, 2026

Large ByteString values (e.g. image payloads) were being fully hex-encoded and logged, causing excessive memory allocation and log output that degraded image processing performance. Now logs just the byte count instead.

…RPC logging

Large ByteString values (e.g. image payloads) were being fully hex-encoded
and logged, causing excessive memory allocation and log output that degraded
image processing performance. Now logs just the byte count instead.

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
@bmc08gt bmc08gt self-assigned this May 4, 2026
@github-actions github-actions Bot added type: fix Bug fix area: network gRPC, connectivity, API, exchange rates labels May 4, 2026
@bmc08gt bmc08gt merged commit e09eca5 into code/cash May 4, 2026
3 checks passed
@bmc08gt bmc08gt deleted the fix/grpc-logging-large-bytestring branch May 4, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: network gRPC, connectivity, API, exchange rates type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant