client: prevent too large RMs #91
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
This prevents RMs that will be too large to send to the server from entering the RMQ, either the RemoteUser.sendRM or the client sendq call sites. The file content subsystem already chunks files appropriately, so that higher level operation already takes into account the max message size. Other operations may need additional checks added, to prevent the RM being created in the first place. fix companyzero#77
alexlyp
approved these changes
Jan 16, 2023
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.
This prevents RMs that will be too large to send to the server from
entering the RMQ, either the RemoteUser.sendRM or the client sendq
call sites.
The file content subsystem already chunks files appropriately, so that
higher level operation already takes into account the max message size.
Other operations may need additional checks added, to prevent the RM
being created in the first place.
fix #77