Skip to content

fix(block-producer): add request timeout to validator sign_block call#2261

Merged
Mirko-von-Leipzig merged 2 commits into
nextfrom
santiagopitte--afix-validator-timeout
Jun 17, 2026
Merged

fix(block-producer): add request timeout to validator sign_block call#2261
Mirko-von-Leipzig merged 2 commits into
nextfrom
santiagopitte--afix-validator-timeout

Conversation

@SantiagoPittella

Copy link
Copy Markdown
Collaborator

Summary

Gives the validator request an explicit timeout so a dropped connection fails.

Changes:

  • BlockProducerValidatorClient::new now takes a timeout and uses .with_timeout() instead of .without_timeout().
  • New DEFAULT_VALIDATOR_TIMEOUT (30s).
  • New --validator.timeout / MIDEN_NODE_VALIDATOR_TIMEOUT CLI flag

Changelog

[[entry]]
scope       = "node"
impact      = "fixed"
description = "Sequencer no longer halts block production for ~15 minutes when the validator connection drops; the `sign_block` call now times out and retries."
[[entry]]
scope       = "node"
impact      = "added"
description = "Added `--validator.timeout` / `MIDEN_NODE_VALIDATOR_TIMEOUT` to configure the request timeout for calls to the validator (default 30s)."

@Mirko-von-Leipzig Mirko-von-Leipzig enabled auto-merge (squash) June 17, 2026 07:11
@Mirko-von-Leipzig Mirko-von-Leipzig merged commit d629e10 into next Jun 17, 2026
23 checks passed
@Mirko-von-Leipzig Mirko-von-Leipzig deleted the santiagopitte--afix-validator-timeout branch June 17, 2026 07:14
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