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

refactor: replace quinn with tokio_rustls #227

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

fabian1409
Copy link
Contributor

No description provided.

@fabian1409 fabian1409 requested a review from dkales October 17, 2024 11:29
mpc-net/src/lib.rs Outdated Show resolved Hide resolved
mpc-net/src/lib.rs Outdated Show resolved Hide resolved
@fabian1409 fabian1409 force-pushed the refactor/mpc_net_tls_stream branch from 79fc52b to e554837 Compare October 18, 2024 09:05
@dkales dkales marked this pull request as ready for review October 18, 2024 10:50
mpc-net/src/lib.rs Outdated Show resolved Hide resolved
@fabian1409 fabian1409 force-pushed the refactor/mpc_net_tls_stream branch 4 times, most recently from e0e0f81 to 34e2b4f Compare October 28, 2024 10:05
@fabian1409 fabian1409 force-pushed the refactor/mpc_net_tls_stream branch 2 times, most recently from e400d11 to e2710be Compare November 7, 2024 15:31
@fabian1409 fabian1409 force-pushed the refactor/mpc_net_tls_stream branch from e2710be to b91f2cd Compare November 25, 2024 09:56
@@ -245,47 +247,27 @@ impl Rep3MpcNet {
let runtime = tokio::runtime::Builder::new_multi_thread()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we want to limit the # of worker threads here to 2 or so?

mpc-net/src/lib.rs Outdated Show resolved Hide resolved
mpc-net/src/lib.rs Outdated Show resolved Hide resolved
@fabian1409 fabian1409 force-pushed the refactor/mpc_net_tls_stream branch from 27c3d60 to 9ca3519 Compare December 16, 2024 13:02
@fabian1409 fabian1409 force-pushed the refactor/mpc_net_tls_stream branch from 9ca3519 to 4bbf970 Compare December 16, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants