Skip to content

Conversation

@kristapsk
Copy link
Member

These ones are currently working ones according both to https://directories.w3ird.tech/ and my local monitoring using jm-dn-uptime.

@AdamISZ
Copy link
Member

AdamISZ commented Oct 17, 2025

Having literally just been looking at this, I sort of half-ACK: 5 seems like a lot? But don't let that stop you if you think this is the right choice :)

edit: 'half-ACK' not just conceptually but as in actually connecting :)

@kristapsk
Copy link
Member Author

It seems that dnodes go down from time to time. I don't see any downsides adding more than less. Am I missing something? My idea now was to just analyze stats and then replace defaults with the most stable known ones in default config.

@AdamISZ
Copy link
Member

AdamISZ commented Oct 17, 2025

It seems that dnodes go down from time to time. I don't see any downsides adding more than less. Am I missing something? My idea now was to just analyze stats and then replace defaults with the most stable known ones in default config.

You're probably not missing anything. I'm just a little worried about "overburdening" by opening so many connections, I don't have a solid (or probably even, correct!) reason for worrying about it. On reflection I guess it is a bit silly, since we end up opening more than 5 connections at the point where we do the actual coinjoin ...

@kristapsk
Copy link
Member Author

#1445

@AdamISZ
Copy link
Member

AdamISZ commented Oct 18, 2025

Fwiw I remembered why I was originally leery of having 5 or more, rather than just 3: it's nice if you're not waiting for the connections, and they don't always work (even if the directory node is working, i.e. Tor connections being flaky), so if you start with only 3 there's a better chance all of the connections work straight away. But, the other side of that is of course the chance of connecting to anyone is higher with 5.

@kristapsk
Copy link
Member Author

This is main blocker for new release. If there are no strong arguments against having 5 directory nodes in default config, I plan to merge this soon.

@AdamISZ
Copy link
Member

AdamISZ commented Oct 30, 2025

tACK the list in d87e7cb across a couple of runs (across several days).

There are 5 in the list, across my two tests, the first time they all connected, the second time I wasn't able to connect to 1 out of 5. But in both cases the application succeeded in working as intended.

@kristapsk kristapsk merged commit 69335d1 into JoinMarket-Org:master Oct 31, 2025
4 of 8 checks passed
@kristapsk kristapsk deleted the update-dnodes branch October 31, 2025 07:25
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