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

Refactor networking module #501

Merged
merged 51 commits into from
Mar 28, 2024
Merged

Refactor networking module #501

merged 51 commits into from
Mar 28, 2024

Conversation

GregHib
Copy link
Owner

@GregHib GregHib commented Mar 23, 2024

  • Reorganise network packaging
  • Combine inventory data with inventories Improve inventory storage disconnect #497
  • Remove login and logout from Player class closes Create Logout Class #438
  • Move encryption library into networking module
  • Remove old/unused classes (ConnectionGatekeeper, ClientManager, LoginManager, NetworkQueue)
  • Add password hash to player info loaded on startup
  • Encrypt password earlier in login stages
  • Make PlayerAccountLoader reusable for bots
  • Stop server on startup or engine exceptions
  • Remove indexing from networking code
  • Remove index allocator
  • Add more networking tests closes Add more tests around GameServer #477

Copy link

codecov bot commented Mar 23, 2024

Codecov Report

Attention: Patch coverage is 76.15176% with 88 lines in your changes are missing coverage. Please review.

Project coverage is 31.36%. Comparing base (0395799) to head (abaa846).

Additional details and impacted files
Files Coverage Δ
.../kotlin/world/gregs/voidps/cache/secure/Huffman.kt 73.63% <100.00%> (ø)
...rld/gregs/voidps/engine/client/EncodeExtensions.kt 0.00% <ø> (ø)
...ps/engine/client/instruction/InstructionHandler.kt 0.00% <ø> (ø)
...s/engine/client/instruction/InstructionHandlers.kt 0.00% <ø> (ø)
...ne/client/instruction/handle/ChatPrivateHandler.kt 0.00% <ø> (ø)
...ine/client/instruction/handle/ChatPublicHandler.kt 0.00% <ø> (ø)
...client/instruction/handle/ChatTypeChangeHandler.kt 0.00% <ø> (ø)
...e/client/instruction/handle/ClanChatJoinHandler.kt 0.00% <ø> (ø)
...e/client/instruction/handle/ClanChatKickHandler.kt 0.00% <ø> (ø)
...e/client/instruction/handle/ClanChatRankHandler.kt 0.00% <ø> (ø)
... and 204 more

... and 388 files with indirect coverage changes

@@             Coverage Diff              @@
##               main     #501      +/-   ##
============================================
+ Coverage     27.20%   31.36%   +4.16%     
- Complexity     4028     4342     +314     
============================================
  Files          1225     1223       -2     
  Lines         39064    39109      +45     
  Branches       8244     8257      +13     
============================================
+ Hits          10626    12268    +1642     
+ Misses        25752    24057    -1695     
- Partials       2686     2784      +98     
Components Coverage Δ
Content 22.34% <30.43%> (-0.13%) ⬇️
Engine 41.42% <71.06%> (+2.35%) ⬆️
Network 71.14% <96.36%> (+58.25%) ⬆️

Copy link

github-actions bot commented Mar 23, 2024

Qodana Community for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

@GregHib GregHib marked this pull request as ready for review March 27, 2024 19:22
@GregHib GregHib merged commit 4f9aaca into main Mar 28, 2024
5 checks passed
@GregHib GregHib deleted the network-tidy branch March 28, 2024 18:15
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.

Add more tests around GameServer Create Logout Class
1 participant