decred / dcrtime

Decred anchored timestamp client, proxy, and server.
ISC License
28 stars 23 forks source link

Retry connection with dcrwallet #65

Open thi4go opened 4 years ago

thi4go commented 4 years ago

This diff adds a retry connection loop when dcrtimed is unable to connect to dcrwallet. It also adds a subsystem logger for dcrtimewallet so that it's able to log on the console.

closes #62

marcopeereboom commented 4 years ago

So this PR needs to be split in two. One for the logging bit and then we need to debate why retrying helps.

thi4go commented 4 years ago

Should we just leave the logging on this one then? @marcopeereboom

marcopeereboom commented 4 years ago

Should we just leave the logging on this one then? @marcopeereboom

Yes. Remove the reconnection logic and create a new issue for that. Then let's debate why we need it and how to implement it. This version seems wrong to me.