Skip to content

Conversation

@dignifiedquire
Copy link
Contributor

Depends on #26

@github-actions
Copy link

github-actions bot commented May 23, 2025

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/net-tools/pr/27/docs/net_tools/

Last updated: 2025-06-05T09:08:30Z

@n0bot n0bot bot added this to iroh May 23, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh May 23, 2025
@matheus23 matheus23 changed the title feat(netmon): epose local addresses on netmon state feat(netmon): expose local addresses on netmon state May 30, 2025
@dignifiedquire dignifiedquire marked this pull request as ready for review June 5, 2025 09:06
@dignifiedquire dignifiedquire requested review from flub and removed request for flub June 5, 2025 09:07
@dignifiedquire dignifiedquire merged commit 51be429 into main Jun 5, 2025
25 of 27 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Jun 5, 2025
@dignifiedquire dignifiedquire deleted the feat-interfaces branch June 5, 2025 11:01
github-merge-queue bot pushed a commit to n0-computer/iroh that referenced this pull request Jun 24, 2025
## Description 

- Use more `Watchable` in  more places
- Drops STUN, ICMP and hairpin probes, they are not actually needed in
our world
- restructure report generation
- Keep QAD connections open, instead of recreating them on every report
- Removes `STUN` functionality from the relay server

**Fixes**

- #3315


## Breaking Changes

- removes `iroh_relay::protos::stun::StunError`
- removes `iroh_relay::server::testing::stun_config`
- removes `iroh_relay::protos::stun`
- removes `iroh_relay::quic::QuicClient::get_addr_and_latency`
- removes `DEFAULT_STUN_PORT`
 


## Depends on 

- [x] #3279 
- [x] n0-computer/net-tools#27

---------

Co-authored-by: Kasey <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants