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

Feat - fragment request server listener #55

Merged
merged 20 commits into from
Jan 6, 2024

Conversation

jabibamman
Copy link
Owner

No description provided.

@jabibamman jabibamman self-assigned this Nov 24, 2023
@jabibamman jabibamman added the enhancement New feature or request label Nov 24, 2023
@jabibamman jabibamman force-pushed the feat/server/fragment-request branch from 0f03ec2 to 7b729c2 Compare November 24, 2023 22:43
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 jabibamman force-pushed the feat/server/fragment-request branch from 937221a to 1641059 Compare November 24, 2023 22:49
@jabibamman
Copy link
Owner Author

- 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 jabibamman force-pushed the feat/server/fragment-request branch from 9d84bcb to 63176e2 Compare November 25, 2023 16:16
@jabibamman jabibamman force-pushed the feat/server/fragment-request branch from c0e52c5 to 73f80c1 Compare December 3, 2023 12:48
@jabibamman jabibamman force-pushed the feat/server/fragment-request branch from 73f80c1 to 13f9255 Compare December 3, 2023 12:55
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 jabibamman force-pushed the feat/server/fragment-request branch 5 times, most recently from bfaa1db to 1ff4b48 Compare December 25, 2023 16:13
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 jabibamman force-pushed the feat/server/fragment-request branch from 1ff4b48 to b8155a3 Compare December 25, 2023 16:17
- 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 jabibamman force-pushed the feat/server/fragment-request branch from 8be538e to 88abbc8 Compare December 26, 2023 15:01
@jabibamman jabibamman changed the title Feat - fragment request server listener (DRAFT) Feat - fragment request server listener Jan 2, 2024
@jabibamman jabibamman requested a review from chikatetsu January 6, 2024 13:00
@jabibamman jabibamman changed the title (DRAFT) Feat - fragment request server listener Feat - fragment request server listener Jan 6, 2024
…request

Restyle Feat - fragment request server listener
@jabibamman jabibamman merged commit 133f180 into develop Jan 6, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants