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.

Sign up for GitHub

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

Jump to bottom

Dev #100

Merged
merged 40 commits into from
Sep 15, 2024
Merged

Dev #100

merged 40 commits into from
Sep 15, 2024

Conversation

MortezaBashsiz
Copy link
Owner

Changelog

  1. cmake integration
  2. clang format
  3. uuid for each connection
  4. use mutex as much as possible
  5. connection timeout implementation
  6. change read method from socket to increase performance
  7. socket shutdown implementation
  8. automate debian package build

SirzechsLucifer666 and others added 30 commits August 28, 2024 17:32
@SirzechsLucifer666
Integrate cmake & reorganize files
7ee1bfc
@SirzechsLucifer666
Add BuildLinux.md & Update README.md
79f583b
@SirzechsLucifer666
Update README.md
3409ee9
@MortezaBashsiz
Merge pull request #83 from SirzechsLucifer666/iss81
f8eb8f4 
Fix #81 & #82 - Integrate CMake & reorganize file structure & Update Build guide
@SirzechsLucifer666
Add .clang-format & .clang-tidy
3fac62c
@SirzechsLucifer666
Add a script to search & format
074347e
@SirzechsLucifer666
Format the files without any logical change
f504096
@MortezaBashsiz
Merge pull request #84 from SirzechsLucifer666/clang-format
ba7730a 
Add Clang format & Clang tidy files
@MortezaBashsiz
Check writeBuffer size before socket write in tcpClient
1c91609
@MortezaBashsiz
Merge pull request #86 from MortezaBashsiz/morteza/check_write_client
2622dd0 
Check writeBuffer size before socket write in tcpClient
@MortezaBashsiz
implement uuid for better log analyz
a3b5004
@MortezaBashsiz
use same uuid in client and connection
ef237e2
@MortezaBashsiz
use mutex in tcpclient
820d640
@MortezaBashsiz
add mutex to config
0d33d03
@MortezaBashsiz
add mutex to server and agent hadlers
53fafe8
@MortezaBashsiz
Merge pull request #89 from MortezaBashsiz/morteza/threads
e4f3f3a 
Morteza/threads
@Ali-Nasrolahi
Timeout implementation for connections.
3a2d5a7 
Objective:
    Automatically closing a connection, which becomes stalled for some reason.

Implementation Details:
    Before every read/write operation, a timer (`deadline_timer`) gets
    started/reseted, and after each successful I/O; the timer gets cancelled.
    When timer's handlers called ,in case of an expiration, it closes associated
    socket with the connection; by doing so, any blocking I/O on respected socket
    will be interrupted.

    This patch only addresses *TCPConnection* and *TCPClient* classes' read and write
    methods. The underlying implementation is identical for both classes.

Fixes #79
@Ali-Nasrolahi
Minor modifications with a few code cleanup.
a72eba9 
Details:
    - Decreasing the timeout value range to 2 bytes.
    - Using seconds instead of miliseconds for timeout indication.
    - Removing unused variable and a couple of variable renaming.
@MortezaBashsiz
Merge pull request #90 from Ali-Nasrolahi/dev-timeout
a125bd5 
Timeout implementation for connections.
@MortezaBashsiz
use transfer_at_least(1) instead of transfer_exactly(1), shutdown soc…
deb303b 
…ket to indicate EOF before close
@MortezaBashsiz
Merge pull request #92 from MortezaBashsiz/morteza/shutdown
c4658c1 
use transfer_at_least(1) instead of transfer_exactly(1), shutdown soc…
@MortezaBashsiz
add socketShutdown function to make it simple and exception handling
92e7b1e
@MortezaBashsiz
Merge pull request #93 from MortezaBashsiz/morteza/socketShutdown
1fe2f15 
add socketShutdown function to make it simple and exception handling
@MortezaBashsiz
add workflow to build debian package
ec91b68
@MortezaBashsiz
add missing rons-on
696ee0c
@MortezaBashsiz
use push method
62a99c7
@MortezaBashsiz
use push path
dd07723
@MortezaBashsiz
fix path
3d723d2
@MortezaBashsiz
add checkout
4bdb5e8
@MortezaBashsiz
Add release
7c03066
@MortezaBashsiz MortezaBashsiz merged commit a355ea4 into main Sep 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MortezaBashsiz @SirzechsLucifer666 @Ali-Nasrolahi @thesudoer2