CaliDog / certstream-go

Go library for connecting to CertStream
MIT License
140 stars 33 forks source link

Replace logging with an error channel #3

Closed Fitblip closed 6 years ago

Fitblip commented 6 years ago

As @mattmoyer kindly pointed out, golang logging is not really a good thing to put in a library like CertStream as there aren't very good facilities for controlling how and when the logger is initialized.

He suggested moving to an API that returns 2 different channels, and having one of them be for errors, which makes a whole lot of sense.

See this thread for more context.