Open
Conversation
These changes follows the structure of the client with a bit of a change. It also has a ServerClient which does just about the same as a Client while exposing a bit of its senders to allow the MasterServer to send messages through it. It also strips down the amount of data that's being stored. We don't need to store the ID, that's tied to the index and the loop already. It's redundant. We also don't need to store the name of the client on the master server. Maybe on the cluster. But on the Master Server we're just redirecting people. It also adds some notes on the approach for the auth server. Finally! :)
- Migrate to using shared::network for ClusterInfo - Import the Security module again - Change ServerClient to MasterClient - Add helper functions for sending data - Improve the Connected event so it takes an ID -
This update seems to have improved the performance of logging by 4x. I haven't done any serious benchmarking but after stress-testing it, I saw noticeable improvements.
- Diagnostics CheckServerVersion was removed since that should be sent before any other command. - Improve enums. - Changed the bind address.
…earch. This also removed ctrlc since we use tokio's ctrlc.
This update also moves config and keys to the data folder.
Member
Author
|
Sustenet v0.1.5 - This project handles all of the updates that should be implemented going forward. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull request changes the structure of the entire app. There was an issue in the way it was designed where it was a little difficult and annoying to interact with it from a Game Engine. I've rewritten the entire app, organized it better into individual structs, and simplified the amount of bytes being stored so RAM usage is just about minimal. The last thing I need to do is implement the Cluster and auth server. I can copy over the internal functionality of each command. I also added some new commands too related to diagnostic and determining how many players there are in a server.
The cluster server will need a permanent connection to the MasterServer so whenever a player requests a list of all servers and information on those servers, the cluster can reply with that information. I'll probably have it to where every 5 seconds, all cluster servers are pinged with a command that asks for information about them.