Skip to content

Add payload logging regression test - #674

Draft
RyanKung wants to merge 1 commit into
masterfrom
codex/payload-log-regression-669
Draft

Add payload logging regression test#674
RyanKung wants to merge 1 commit into
masterfrom
codex/payload-log-regression-669

Conversation

@RyanKung

Copy link
Copy Markdown
Member

Summary

  • route oversized-payload diagnostics through a metadata-only logging helper
  • add a tracing regression test proving custom message body text is not emitted
  • keep tx id, message kind, byte count, and limit diagnostics intact

Validation

  • cargo +nightly fmt --all --check
  • cargo test -p rings-core oversized_payload_log_omits_message_body --features dummy -- --nocapture
  • cargo test -p rings-core malformed_outbound_payload_is_rejected_before_connection_admission --features dummy
  • git diff --check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant