Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Relax OnionMessenger::peer_disconnected assertion #2787

Merged
merged 1 commit into from
Dec 12, 2023

Commits on Dec 12, 2023

  1. Relax OnionMessenger::peer_disconnected assertion

    When a peer is connected, OnionMessenger tracks it only if it supports
    onion messages. On disconnect, we debug_assert that the peer was in a
    state ConnectedPeer, failing when it is in the PendingConnection state.
    However, we were mistakenly asserting for peers that we were not
    tracking (i.e., that don't support onion messages). Relax the check to
    not fail on the latter.
    jkczyz committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    a4f2c36 View commit details
    Browse the repository at this point in the history