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

Fix docker for relay_v1 and prepare for relay_v2 #6

Open
wants to merge 8 commits into
base: dcutr
Choose a base branch
from

Conversation

r-zig
Copy link

@r-zig r-zig commented Nov 4, 2021

One of the changes was from ipv6 to ipv4 since I notice that ipv6 is not fully supported and I want the example to be as simple as possible to beginner.
I also prepared the Docker to support docker compose of both relay versions.

@mxinden mxinden changed the base branch from dcutr to relay-v2 November 15, 2021 13:03
@mxinden mxinden changed the base branch from relay-v2 to dcutr November 15, 2021 13:04
@mxinden
Copy link
Owner

mxinden commented Nov 26, 2021

Sorry for the delay here @r-zig. I cherry-picked the command line parsing changes to the relay-v2 and dcutr branch. I still need to think a bit more about how to structure the examples between relay-v2 and dcutr. Maybe I will write a larger hole punching example.

Anyways, thanks for the patches here. Have you been able to test hole punching yourself?

@r-zig
Copy link
Author

r-zig commented Dec 3, 2021

Sorry @mxinden - I didn't have enough time to check yet. Some busy times. Hope to return and check it next during the month.

mxinden pushed a commit that referenced this pull request Sep 5, 2022
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