nictuku / dht

Kademlia/Mainline DHT node in Go.
Other
826 stars 144 forks source link

remove glog completely #65

Closed cenkalti closed 5 years ago

cenkalti commented 5 years ago

I have forgot some to remove some glog imports on #62.

This PR removes all remaining glog imports and replaces logging calls to recently introduced DebugLogger.

I have removed 3 log calls because it was not easy to pass the logger to those functions. They didn't look critical to me. @nictuku If you want I can try to find an another solution.

To recall the issue; glog registers its own flags so they interfere with application's flags when dht is used as a library. See: https://github.com/golang/glog/blob/23def4e6c14b4da8ac2ed8007337bc5eb5007998/glog.go#L398-L404