Skip to content

Conversation

@JavaSaBr
Copy link
Owner

@JavaSaBr JavaSaBr commented Oct 20, 2025

Update network module:

  1. Add config settings to define the maximum possible network packet size
  2. Remove the doubled write buffer for network writers
  3. Fix stack overflow for the server network packet reader under high load

@JavaSaBr JavaSaBr self-assigned this Oct 20, 2025
@github-actions
Copy link

Overall Project 36.49% -0.12%
Files changed 77.71% 🍏

File Coverage
DefaultSslNetworkPacketWriter.java 100% 🍏
IdBasedNetworkPacketWriter.java 100% 🍏
DefaultNetworkPacketWriter.java 100% 🍏
ServerNetworkConfig.java 91.3% 🍏
NetworkConfig.java 87.88% 🍏
DefaultServerNetwork.java 73.87% 🍏
AbstractNetworkPacketWriter.java 61.54% -4.9% 🍏
AbstractNetworkPacketReader.java 58.42% -0.91%

@github-actions
Copy link

Overall Project 36.6% -0.24%
Files changed 70.85% 🍏

File Coverage
MalformedProtocolException.java 100% 🍏
DefaultSslNetworkPacketWriter.java 100% 🍏
IdBasedNetworkPacketWriter.java 100% 🍏
DefaultNetworkPacketWriter.java 100% 🍏
ServerNetworkConfig.java 91.3% 🍏
NetworkConfig.java 87.88% 🍏
DefaultServerNetwork.java 73.87% 🍏
AbstractNetworkPacketWriter.java 61.54% -4.9% 🍏
AbstractNetworkPacketReader.java 60.82% -4.2%

@JavaSaBr JavaSaBr merged commit d91547f into develop Oct 20, 2025
6 checks passed
@JavaSaBr JavaSaBr deleted the extend-network-module branch October 20, 2025 17:14
@github-actions
Copy link

Overall Project 36.57% -0.24%
Files changed 70.85% 🍏

File Coverage
MalformedProtocolException.java 100% 🍏
DefaultSslNetworkPacketWriter.java 100% 🍏
IdBasedNetworkPacketWriter.java 100% 🍏
DefaultNetworkPacketWriter.java 100% 🍏
ServerNetworkConfig.java 91.3% 🍏
NetworkConfig.java 87.88% 🍏
DefaultServerNetwork.java 73.87% 🍏
AbstractNetworkPacketWriter.java 61.54% -4.9% 🍏
AbstractNetworkPacketReader.java 59.75% -4.2%

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