Skip to content
This repository has been archived by the owner on Jun 5, 2021. It is now read-only.

cmd: added NAT traversal POC. #38

Merged
merged 1 commit into from Jun 13, 2018
Merged

cmd: added NAT traversal POC. #38

merged 1 commit into from Jun 13, 2018

Conversation

backkem
Copy link
Contributor

@backkem backkem commented Jun 10, 2018

Related to #34, I created a little NAT traversal demo to get more familiar with the process. What I noticed:

  • The open connection has to be re-used while changing the remote IP in order to keep the local address and the port assigned by the NAT stable while switching to communication with the peer. This doesn't work when using Dial to setup the connection. In addition, the STUN and peer traffic has to be multiplexed on the same connection. I think we'll have to revise the client implementation to take this into account.
  • We should investigate ways to test NAT traversal since it requires 2 machines behind a NAT and a third to run the STUN server. Maybe some sort of Docker/VirtualBox setup.

@backkem backkem requested a review from ernado June 10, 2018 13:59
@ernado ernado merged commit d4ce27c into gortc:master Jun 13, 2018
@backkem backkem deleted the traversal branch June 29, 2018 17:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants