Skip to content

Conversation

@etorreborre
Copy link
Contributor

@etorreborre etorreborre commented Jan 26, 2026

The PR makes sure that we can connect to an upstream node if:

  • It is not yet started when we start.
  • It is restarted while we are connected.

In order to do this:

  • The reading/writing stages in the Muxer are supervised.
  • The Muxer itself is supervised by the Connection which closes the loop since the Connection is supervised by the Manager.
  • FetchBlock errors are not considered as fatal anymore since they can happen on disconnection.
  • Similarly a rollback can fail on reconnection if the corresponding block was not yet downloaded
    -> TODO: what guarantees that we will eventually download the block?

In addition:

  • Two tests were added to support the 2 disconnection scenarii presented earlier.
  • The tests were refactored since the setup and the assertions contains more code now.
  • The Muxer has a role field that is used to annotate traces, which is useful when reading log messages.
  • The connection timeout that was hard-coded in the Manager is now part of its configuration.

@etorreborre etorreborre self-assigned this Jan 26, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@etorreborre
Copy link
Contributor Author

@CodeRabbit review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

…n the responder is started

Signed-off-by: etorreborre <etorreborre@yahoo.com>
@etorreborre etorreborre force-pushed the etorreborre/fix/reconnection-to-upstream branch from 3f24d68 to c0eff15 Compare January 26, 2026 13:07
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.

2 participants