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

Add steamworks transport #89

Merged
merged 30 commits into from
Jul 18, 2023

Conversation

TrustNoOneElse
Copy link
Contributor

@TrustNoOneElse TrustNoOneElse commented Jun 17, 2023

Target of this pr is to add SteamworkNetworkingSockets as a alternative transport.

This might require some changes on internal existing code, before breaking-changes are introduced in this pr will the maintainer be notified and possiblities discussed.

This has the advantage to have an existing relay system for P2P games, like coop games, the advantage is that the ip will not be revealed if its used in this context.

It has also the advantage that packages are already encrypted.

Fell always free to review code in the early Stage, it helps to discuss/correct this before further implemented and takes more time to actually change it.

Goals:

  • Configure demo to show the new transport or alternativly if not other possible create a new demo
  • Add Bevy support for this transport
  • Add a readme
  • Working transport based on SteamworkNetworkingSockets for Clients
  • The structure should support that SteamGameServer can be later added or be added if its possible

@TrustNoOneElse TrustNoOneElse marked this pull request as ready for review July 10, 2023 17:38
@lucaspoffo lucaspoffo self-requested a review July 11, 2023 00:34
@lucaspoffo lucaspoffo merged commit c67a219 into lucaspoffo:master Jul 18, 2023
@lucaspoffo
Copy link
Owner

lucaspoffo commented Jul 18, 2023

Thanks for the PR, did some small tweaks on the steam transport, mostly unifying some functions and making it more like the style I use.

Also removed the demos and bevy_renet changes, the scope was too big, and is gonna take a while to get these ones right. Let's remove them for now and merge the transport for now.

Added an echo example for renet_steam, which is pretty much the same one from renet/examples/echo.rs but using the steam transport.

🎉 🎉

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