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

Max/echo server clap #5317

Draft
wants to merge 102 commits into
base: develop
Choose a base branch
from
Draft

Max/echo server clap #5317

wants to merge 102 commits into from

Conversation

mfahampshire
Copy link
Contributor

@mfahampshire mfahampshire commented Jan 8, 2025 •
edited
Loading

TODO

  • merge Max/client pool #5188
  • rebase develop
  • make utils exportable + cleanup from
    • echo server
    • internal tool
  • echo server lib readme list
  • does it need to be multithread to speed it up?
  • fix tcpproxyserver FFI

This change is Reviewable

@mfahampshire
tcp conn tracker
c406814
@mfahampshire
small tweak to tcp tracker
f53d2aa
@mfahampshire
name change + specific inc and dec logging
1130d0f
@mfahampshire
changed logging
d423cde
@mfahampshire
tweak
0c36e64
@mfahampshire
tweak
109de83
@mfahampshire
tweak existing conn tracker logic
2e8e9a4
@mfahampshire
make default decay const
530d744
@mfahampshire
first pass connpool
5464829
@mfahampshire
fix flipped lowerthan waiting for pool
8e8eceb
@mfahampshire
first pass disconnect
415c30f
@mfahampshire
trying to get disconnect to remove from pool
6f4421d
@mfahampshire
remove double accounting for moment
5083b42
@mfahampshire
remove comments and reduce logging verbosity
4519bd6
@mfahampshire
comments
699aa61
@mfahampshire
removed a bunch of commenting from example
4273959
@mfahampshire
removed a bunch of commenting from example
8fcea5f
@mfahampshire
err handling conpool start
4bb05f3
@mfahampshire
added notes for next features
8c8e085
@mfahampshire
first version working
c49f7f9
@mfahampshire
first pass spin out client_pool
f87f57b
@mfahampshire
cancel token
4f1bac2
@mfahampshire
logging change
13a6f0b
@mfahampshire
minor tweaks
56becc3
@mfahampshire
bump default decay time
4eccc59
@mfahampshire
updated code commnet
c8ca174
@mfahampshire
bump default decay time
95274e3
@mfahampshire
bump default decay time (again)
7563ccc
@mfahampshire
temp commit
a0e55f4
@mfahampshire
add duplicate packets received to troubleshooting
731f69a
@mfahampshire
fix test
60e8f82
@mfahampshire
remove active conn
2d7957a
@mfahampshire
logging tweak
eef3405
@mfahampshire
work on test
6606fc8
@mfahampshire
working on script fr testing all
1cd42bd
@mfahampshire
minor log tweak
ec77f1e
@mfahampshire
fixed test: needed proxiedMessage format
0337c04
@mfahampshire
first pass tester
ffcc4e8
@mfahampshire
cargo + todo list
49e1134
@mfahampshire
cancel token
5e9d253
@mfahampshire
remove scratch
a87753f
@mfahampshire
remove runs
20697fd
@mfahampshire
add dir creation + time dir to results
54cec26
@mfahampshire
cancel token fixes ongoing
376eca6
@mfahampshire
added test start fn to client_pool: uses specified gw for all clients
93f063b
@mfahampshire
comment
8dda5fb
@mfahampshire
debugging failing lock quiring
54d7b37
@mfahampshire
re-add commented out code fr sharing
50191cf
@mfahampshire
remove comments
3e1ea8e
@mfahampshire
comments
378ed88
@mfahampshire
rearrange code, comment for rewrite of run()
b73d66e
@mfahampshire
trying to make run_with_shutdown killable
d86286e
@mfahampshire
first pass done
e638f87
@mfahampshire
update lock
3f90de1
@mfahampshire
proper disconnect signals
45617cd
@mfahampshire
proper disconnect signal management
5b536ee
@mfahampshire
license
499b1c7
@mfahampshire mfahampshire requested a review from octol as a code owner January 8, 2025 15:27
@mfahampshire mfahampshire marked this pull request as draft January 8, 2025 15:27
@vercel Vercel
Copy link

vercel bot commented Jan 8, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-nextra ⬜️ Ignored (Inspect) Jan 8, 2025 3:30pm
nym-next-explorer ⬜️ Ignored (Inspect) Jan 8, 2025 3:30pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@octol octol Awaiting requested review from octol octol is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mfahampshire