-
Notifications
You must be signed in to change notification settings - Fork 48
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
P2P Connection Helper & config adjustments #18
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Changed the config. Now the SIT client determines the hosting port.
…T.Aki-Server-Mod into nat-punchthrough
- Added new route /coop/server/connectionInfo to get and set the server connection info. - P2PConnectionHelper is now NatPunchHelper and is only used for nat punching. - CoopMatch now holds the server type, nat, ip and port information.
Removed /coop/server/connectionInfo endpoints as they are no longer used. Connection information will now be provided at CreateMatch and JoinMatch instead.
- NatPunchHelper is now NatHelper. - NatHelper is a websocket relay server that provides the server public endpoints to clients. - Ip, port and nat type is no longer stored in CoopMatch. All is handled by NatHelper, except for ServerType. - ServerType is stored in CoopMatch in order to provide it to CoopGame (determine whether we should create a GameClient/GameServer TCP/UDP).
This should return the host's local IP (if you connected to SPT using a local IP) or the public IP (if you connected to SPT using a public IP).
- Added dependencies for nat-upnp module - Added port to coop match - NatHelper is now using the port in coop match to provide the remote address obtained by the web socket connection, thus removing the need to add a bogus remote endpoint by the client. - Cleanup some unused variables
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.