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

Initial configuration doesn't work #24

Closed
kaotisk-hund opened this issue May 5, 2022 · 6 comments
Closed

Initial configuration doesn't work #24

kaotisk-hund opened this issue May 5, 2022 · 6 comments

Comments

@kaotisk-hund
Copy link
Collaborator

In the generated config that ships with cjdns-installer, all the connections are getting to the IPv4 section.

There are also a lot nodes with too much information (known bug) in their blocks as well as others no connections since 2015.

@andrasfuchs
Copy link
Collaborator

andrasfuchs commented May 9, 2022

I think the configuration file problem must be related to the issue that cjdns has: running the cjdroute --genconf >> cjdroute.conf command will produce an invalid file. To fix it, the line #117 must be removed from the cjdroute.conf file that is installed into the c:\Program Files (x86)\cjdns\ folder by default.
image
We can't do more about it until they do a fix in the original cjdns repo.

Does having too many peers in our config file cause problems for cjdns? My understanding is that we need them only for the initialization phase and they will be updated/removed as soon as we find valid peers. I added a few more recent peers from Hyperbolia peers repo, but they might be already out of date. fc00 seems to be down too.
I agree that the no longer working peers should be removed from public_peers.txt.

@kaotisk-hund
Copy link
Collaborator Author

About the "too many peers". I think there is no problem with the amount of them. They problem from peering information is that when we mention a connection, cjdns waits an "up to number" bytes. Important information is:

  • login ( if not default )
  • password
  • publickey
  • peerName (optional)
    the rest is either way ignored or choking cjdns configuration file. However, if cjdns receives more bytes can lead to problems. Sorry I don't have an issue link or a point to show to. I know this by testing and discussion over irc.

@kaotisk-hund
Copy link
Collaborator Author

I am over testing which ones are working and preparing a first stage patch over the same file.
I will then introduce a "pubic_peers_6.txt" file moving the IPv6 peers there.

@kaotisk-hund
Copy link
Collaborator Author

We can't do more about it until they do a fix in the original cjdns repo.

which version do we have? 21.1.6 seems to have no issue with that.

@andrasfuchs
Copy link
Collaborator

Great, thank you!

I think we need to modify the addPublicPeers.vbs and installer.nsi files to add the ipv6 peers from the new public_peers_6.txt.
Also, if we have them in separate files, it would make sence to name them public_peers_ipv4.txt and public_peers_ipv6.txt, I think.

We have cjdns v21.2 at the moment. Do you think we should roll back to v21.1?

@kaotisk-hund
Copy link
Collaborator Author

I think it 's finally ready. I can't test it now, cause I can't build it :(

Also, sorry for not answering about the roll back. For development it's not that of an issue, for shipping a binary installer ready to go, I think either a roll back or a surgical removal would be enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants