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: add TorV3 support #66

Conversation

oleonardolima
Copy link
Contributor

No description provided.

@oleonardolima oleonardolima changed the title Feat/add tor connection support feat: add TorV3 support Aug 8, 2024
@rustaceanrob
Copy link
Owner

Ran this locally and LGTM. Is this ready to go?

@oleonardolima
Copy link
Contributor Author

Ran this locally and LGTM. Is this ready to go?

Did you manage to connect to an onion address peer ?

I didn't get any yet, might've broken something locally.

@rustaceanrob
Copy link
Owner

The example uses a preset peer and only connects to one peer. I'll change it locally to try to find one at random 😂

Copy link
Owner

@rustaceanrob rustaceanrob left a comment

Choose a reason for hiding this comment

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

Got the bug. Was able to connect to onion addrs!

src/peers/tor.rs Show resolved Hide resolved
@oleonardolima oleonardolima force-pushed the feat/add-tor-connection-support branch from 6e78073 to 2ddd0f2 Compare August 9, 2024 10:27
@rustaceanrob
Copy link
Owner

Works. OK if I merge?

@rustaceanrob rustaceanrob marked this pull request as ready for review August 9, 2024 14:17
@rustaceanrob rustaceanrob merged commit 022d0e7 into rustaceanrob:master Aug 9, 2024
12 checks passed
@oleonardolima oleonardolima deleted the feat/add-tor-connection-support branch August 9, 2024 14:48
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