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

Change logging #55

Closed
whyrusleeping opened this issue Sep 12, 2014 · 5 comments
Closed

Change logging #55

whyrusleeping opened this issue Sep 12, 2014 · 5 comments
Labels
help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature

Comments

@whyrusleeping
Copy link
Member

I would like to update all the logging in ipfs (util.DOut, and friends) to instead use go-logging. This will help us out immensely once we start doing larger deloys and things start failing.

@whyrusleeping whyrusleeping added kind/enhancement A net-new feature or improvement to an existing feature help wanted Seeking public contribution on this issue ready labels Sep 12, 2014
@whyrusleeping
Copy link
Member Author

This is quite related to the work @perfmode is doing in #66, updating for reference

@btc
Copy link
Contributor

btc commented Sep 21, 2014

How does go-logging compare to glog?

@whyrusleeping
Copy link
Member Author

glog looks pretty nice. I like the 'V' feature it has. One thing i like better about go-logging is the ability to set different logging levels per module. i.e. when debugging the dht, you can set all dht logging levels on, but turn ones related to other package off (or at least to Error only)

@jbenet
Copy link
Member

jbenet commented Sep 21, 2014

That's really useful—
Sent from Mailbox

On Sun, Sep 21, 2014 at 11:33 AM, Jeromy Johnson notifications@github.com
wrote:

glog looks pretty nice. I like the 'V' feature it has. One thing i like better about go-logging is the ability to set different logging levels per module. i.e. when debugging the dht, you can set all dht logging levels on, but turn ones related to other package off (or at least to Error only)

Reply to this email directly or view it on GitHub:
#55 (comment)

@whyrusleeping
Copy link
Member Author

closing as we are using almost exclusively go-logging now

@Stebalien Stebalien mentioned this issue May 26, 2020
77 tasks
laurentsenta pushed a commit to laurentsenta/kubo that referenced this issue Feb 25, 2022
…/multiformats/go-multiaddr-0.1.2

Bump github.com/multiformats/go-multiaddr from 0.1.1 to 0.1.2
laurentsenta pushed a commit to laurentsenta/kubo that referenced this issue Feb 25, 2022
…dules/github.com/multiformats/go-multiaddr-0.1.2

Bump github.com/multiformats/go-multiaddr from 0.1.1 to 0.1.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Seeking public contribution on this issue kind/enhancement A net-new feature or improvement to an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants