Skip to content

Tunneling UDP in addition to TCP #72

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

radiumatic
Copy link

Hi,
I cobbled together something to allow forwarding UDP packets using the unreliable datagrams feature of Iroh.
It works, I think.

PS: I tried connecting using different nodes to the same listen-udp node but it didn't really work. I'm not sure why. I also couldn't get it to produce deterministic NodeIDs.

radiumatic and others added 2 commits July 27, 2025 19:26
The UDP socket not properly forwarding the packets on the client side
@n0bot n0bot bot added this to iroh Jul 27, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Jul 27, 2025
@flub
Copy link

flub commented Jul 30, 2025

Hi @radiumatic, thanks for contributing! Did you see #53 ? Is your approach substantially different?

@radiumatic
Copy link
Author

To be honest I'm not really good at Rust, but it doesn't look like there are any large differences. Skimming through #53 , the thing I noticed was it using more clones and Arcs than my PR. It also can create more than one socket to the local, actual server. So maybe my PR is a bit faster?

We both modified some commands to output more information, like the actual port being listened on, I think. And this PR just chooses a conservative MTU value and sends everything through that.

I'm sorry if this is a bit unrelated, but can the relay/signaling server implementation sit entirely behind a reverse proxy? Is a "client" and "server" model (with multiple nodes connecting to one) not working intended behavior?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

2 participants