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.

Sign up for GitHub

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

Jump to bottom

fix(iroh-net)!: only call quinn_connect if a send addr is available #2225

Merged
merged 6 commits into from
Apr 26, 2024
Merged

fix(iroh-net)!: only call quinn_connect if a send addr is available #2225

merged 6 commits into from
Apr 26, 2024

Commits on Apr 26, 2024

  1. fix(iroh-net): only call quinn_connect if a send addr is available

    @Frando @ramfox
    Frando authored and ramfox committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    1b3fe7c View commit details
    Browse the repository at this point in the history

  2. move run_dns_and_pkarr_servers to iroh::net::test_utils 

    to properly test that we have fixed the discovery bug, we need to run the test dns and pkarr relay servers, so they should be moved to `test_utils` so the code can be shared
    @ramfox
    ramfox committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    397c985 View commit details
    Browse the repository at this point in the history

  3. write test that ensures we launch discovery if we do not have address… 

    … info for a given node_id
    @ramfox
    ramfox committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    71f6cca View commit details
    Browse the repository at this point in the history

  4. refactor, remove some magicsock apis and add 

    this method handles the logic of getting a mapped quic addr and launching discovery if it is needed
    @ramfox
    ramfox committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    d034423 View commit details
    Browse the repository at this point in the history

  5. feat!(iroh-net): NodeInfo.relay_url is now a RelayUrlInfo 

    `RelayUrlInfo` combines the `RelayUrl` with any "aliveness" and
latency information we have. This is propgated up into the
`ConnectionInfo`.
    @ramfox
    ramfox committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    55cfc91 View commit details
    Browse the repository at this point in the history

  6. zero out non-deterministic fields in test

    @ramfox
    ramfox committed Apr 26, 2024
    Configuration menu
    Copy the full SHA
    758d0ad View commit details
    Browse the repository at this point in the history