-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat - fragment request server listener #55
Merged
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
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
November 24, 2023 22:43
0f03ec2
to
7b729c2
Compare
This commit adds Rustdoc comments to the `serialize_request`, `deserialize_request`, and `deserialize_message` functions in the `serialization.rs` file. It also updates an import statement in the `handler.rs` file to fix a compilation error.
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
November 24, 2023 22:49
937221a
to
1641059
Compare
- Updated client code to print messages with a prefix indicating it is from the client - Updated server code to print messages with a prefix indicating it is from the server - Modified server code to handle client connection and read data from the client - Added logic in server code to create a task for a received request and serialize it as JSON - Implemented serialization of FragmentTask in server code
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
November 25, 2023 16:16
9d84bcb
to
63176e2
Compare
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
December 3, 2023 12:48
c0e52c5
to
73f80c1
Compare
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
December 3, 2023 12:55
73f80c1
to
13f9255
Compare
This commit sends the serialized task to the client at the peer address.
- Updated the `deserialize_message` function in `server/src/messages/serialization.rs` to use the `FragmentRequest::deserialize` method instead of `deserialize_request`. - Added debug logs to track the deserialization process. - Modified the error handling in `FragmentRequest::deserialize` to check for the presence of a valid "FragmentRequest" object in the JSON response.
jabibamman
force-pushed
the
feat/server/fragment-request
branch
5 times, most recently
from
December 25, 2023 16:13
bfaa1db
to
1ff4b48
Compare
This commit adds optional flags to the CliClientArgs struct in the parser module. The new flags are "verbose", "debug", and "open" with default values set to false. These flags can be used to enable or disable logging, debug mode, and opening the browser respectively.
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
December 25, 2023 16:17
1ff4b48
to
b8155a3
Compare
- Read total size and JSON size from the stream. - Read the buffer with the specified total size. - Convert the buffer to a UTF-8 string and handle invalid sequences. - Debug log received JSON data. - Deserialize the received data into a message object.
jabibamman
force-pushed
the
feat/server/fragment-request
branch
from
December 26, 2023 15:01
8be538e
to
88abbc8
Compare
jabibamman
changed the title
Feat - fragment request server listener
(DRAFT) Feat - fragment request server listener
Jan 2, 2024
jabibamman
changed the title
(DRAFT) Feat - fragment request server listener
Feat - fragment request server listener
Jan 6, 2024
chikatetsu
approved these changes
Jan 6, 2024
…request Restyle Feat - fragment request server listener
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.
No description provided.