README indicates errant default behavior

[UsmannK]

README.md reads "To try utreexo you do not need to run a server as we have a server set up for testing purposes which the client connects to by default." However, the client attempts to connect to 127.0.0.1:8338 on startup.

The code responsible is in csn/config.go:

utreexo/csn/config.go

Lines 103 to 111 in 259fc1a

	// if no host was given, default to localhost
	if *remoteHost == "" {
	cfg.remoteHost = "127.0.0.1:8338"
	} else {
	if !strings.ContainsRune(*remoteHost, ':') {
	str := *remoteHost + ":8338"
	cfg.remoteHost = str
	}
	}

Should the README be changed, or the code?

Thanks!

[kcalvinalvin]

Hello!

It's the readme that should be changed. I think the line

"To try utreexo you do not need to run a server as we have a server set up for testing purposes which the client connects to by default."

was written back when we did the utreexo v1 release. Since then, there have been breaking changes so the code was changed to default to the local server.

The v1 client (found in the tags section of the repo) should still work properly.

[DiptoChakrabarty]

Can I make this change in the ReadMe

[kcalvinalvin]

Can I make this change in the ReadMe

Yes. No need to ask permissions :)