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

chore(config): adapt default directory nodes #126

Merged
merged 3 commits into from
Sep 16, 2024

Conversation

theborakompanioni
Copy link
Collaborator

@theborakompanioni theborakompanioni commented May 16, 2024

Note: Still a draft - do not merge. Blocked: Waiting for feedback.


PR based on this comment JoinMarket-Org/joinmarket-clientserver#1445 (comment) (similar to #112)

add

Adds node (comment)

  • odpwaf67rs5226uabcamvypg3y4bngzmfk7255flcdodesqhsvkptaid.onion:5222

replace

Replaces (defunct?) node (comment)

  • wkd3kd73ze62sqhlj2ebwe6fxqvshw5sya6nkvrgcweegt7ljhuspaid.onion:5222

with

  • plq5jw5hqke6werrc5duvgwbuczrg4mphlqsqbzmdfwxwkm2ncdzxeqd.onion

@theborakompanioni
Copy link
Collaborator Author

theborakompanioni commented Sep 16, 2024

From JoinMarket-Org/joinmarket-clientserver#1445 (comment)

Current state of directory nodes:
g3hv4uynnmynqqq2mchf3fcm3yd46kfzmcdogejuckgwknwyq5ya6iad.onion:5222
3kxw6lf5vf6y26emzwgibzhrzhmhqiw6ekrek3nqfjjmhwznb2moonad.onion:5222 🆗
bqlpq6ak24mwvuixixitift4yu42nxchlilrcqwk2ugn45tdclg42qid.onion:5222 🆗
wkd3kd73ze62sqhlj2ebwe6fxqvshw5sya6nkvrgcweegt7ljhuspaid.onion:5222
ylegp63psfqh3zk2huckf2xth6dxvh2z364ykjfmvsoze6tkfjceq7qd.onion:5222
plq5jw5hqke6werrc5duvgwbuczrg4mphlqsqbzmdfwxwkm2ncdzxeqd.onion:5222
odpwaf67rs5226uabcamvypg3y4bngzmfk7255flcdodesqhsvkptaid.onion:5222 🆗

So.. suggestion is to just include odpwaf6.. in addition to the default nodes. Reasonable?

So, new list is:
g3hv4uynnmynqqq2mchf3fcm3yd46kfzmcdogejuckgwknwyq5ya6iad.onion:5222 ❌ default
3kxw6lf5vf6y26emzwgibzhrzhmhqiw6ekrek3nqfjjmhwznb2moonad.onion:5222 🆗 default
bqlpq6ak24mwvuixixitift4yu42nxchlilrcqwk2ugn45tdclg42qid.onion:5222 🆗 default
odpwaf67rs5226uabcamvypg3y4bngzmfk7255flcdodesqhsvkptaid.onion:5222 🆗 non-default

via

curl -v -x socks5h://localhost:9050 -s <onion_url>:5222

@theborakompanioni theborakompanioni marked this pull request as ready for review September 16, 2024 10:28
@openoms
Copy link

openoms commented Sep 16, 2024

Makes sense, can confirm the last three working with the simple script from: openoms/joininbox#108

directory_nodes_list="g3hv4uynnmynqqq2mchf3fcm3yd46kfzmcdogejuckgwknwyq5ya6iad.onion 3kxw6lf5vf6y26emzwgibzhrzhmhqiw6ekrek3nqfjjmhwznb2moonad.onion bqlpq6ak24mwvuixixitift4yu42nxchlilrcqwk2ugn45tdclg42qid.onion odpwaf67rs5226uabcamvypg3y4bngzmfk7255flcdodesqhsvkptaid.onion"
echo $directory_nodes_list
for i in $directory_nodes_list; do
  time torsocks nc -zv $i 5222
done
g3hv4uynnmynqqq2mchf3fcm3yd46kfzmcdogejuckgwknwyq5ya6iad.onion 3kxw6lf5vf6y26emzwgibzhrzhmhqiw6ekrek3nqfjjmhwznb2moonad.onion bqlpq6ak24mwvuixixitift4yu42nxchlilrcqwk2ugn45tdclg42qid.onion odpwaf67rs5226uabcamvypg3y4bngzmfk7255flcdodesqhsvkptaid.onion
1726483037 ERROR torsocks[169706]: Host unreachable (in socks5_recv_connect_reply() at socks5.c:539)
nc: connect to g3hv4uynnmynqqq2mchf3fcm3yd46kfzmcdogejuckgwknwyq5ya6iad.onion (127.42.42.0) port 5222 (tcp) failed: No route to host

real	0m8.209s
user	0m0.006s
sys	0m0.000s
Connection to 3kxw6lf5vf6y26emzwgibzhrzhmhqiw6ekrek3nqfjjmhwznb2moonad.onion (127.42.42.0) 5222 port [tcp/xmpp-client] succeeded!

real	0m0.368s
user	0m0.002s
sys	0m0.006s
Connection to bqlpq6ak24mwvuixixitift4yu42nxchlilrcqwk2ugn45tdclg42qid.onion (127.42.42.0) 5222 port [tcp/xmpp-client] succeeded!

real	0m0.355s
user	0m0.005s
sys	0m0.015s
Connection to odpwaf67rs5226uabcamvypg3y4bngzmfk7255flcdodesqhsvkptaid.onion (127.42.42.0) 5222 port [tcp/xmpp-client] succeeded!

real	0m2.022s
user	0m0.015s
sys	0m0.005s

@theborakompanioni
Copy link
Collaborator Author

Makes sense, can confirm the last three working with the simple script from: openoms/joininbox#108

Thanks for verifying 🙏
Merging!

@theborakompanioni
Copy link
Collaborator Author

A user reported ylegp appears to be working sometimes. Add?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concept enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants