Skip to content

Better submit block relay error logging - #502

Merged
JasonVranek merged 3 commits into
mainfrom
fix/v2-submit-block-relay-error
Sep 29, 2026
Merged

JasonVranek merged 3 commits into
mainfrom
fix/v2-submit-block-relay-error

Conversation

@JasonVranek

Copy link
Copy Markdown
Collaborator

The v2 branch added in #468 added a logging regression that dropped the relay's reason in the body. Now when a relay refuses a submit_blinded_block, log the relay's explanation instead of just a status code.

Note passing CI required bumping the rustls version

@JasonVranek
JasonVranek requested review from a team, ManuelBilbao and ltitanb September 28, 2026 17:26
@JasonVranek
JasonVranek force-pushed the fix/v2-submit-block-relay-error branch from 10795d8 to 9040c18 Compare September 28, 2026 17:49
@JasonVranek
JasonVranek force-pushed the fix/v2-submit-block-relay-error branch from 9040c18 to 600889c Compare September 29, 2026 20:09
@JasonVranek
JasonVranek merged commit 02cca17 into main Sep 29, 2026
4 checks passed
@JasonVranek
JasonVranek deleted the fix/v2-submit-block-relay-error branch September 29, 2026 20:09
JasonVranek added a commit that referenced this pull request Oct 2, 2026
…secrets, rustls bump

Brings in #502 (submit-block relay error logging), the v0.11.0-rc2/rc3
releases with the rustls advisory bump, relay headers read from a secret
file or env var (#498), and the ws stream metrics endpoint (#499).

Conflict resolutions:
- relay.rs: kept epbs's relay_headers() helper, shared by the HTTP client
  and the stream handshake, and moved main's header resolution into it
  (HeaderSource literal/file/env, sensitive values, the secret-source log
  line). Its error messages now format the key, which epbs's did not.
- config/pbs.rs: one test module holding epbs's unknown-[pbs]-field tests
  and main's header-source tests.
- config/utils.rs: main's RELAY_URL test constant in test_env.
- Cargo.lock: main's lockfile, plus only what epbs adds.
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.

3 participants