search

majestrate / XD

meme bittorrent client
https://xd-torrent.github.io/
MIT License
241 stars 25 forks source link

Logs are spammed with speed info #23

Closed l-n-s closed 7 years ago

l-n-s commented 7 years ago

Can it be optional?

[NFO] 2017-09-15 11:22:23.676528458 -0400 EDT m=+6430.255593282 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec
[NFO] 2017-09-15 11:22:33.676941919 -0400 EDT m=+6440.256006044 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec
[NFO] 2017-09-15 11:22:43.677353704 -0400 EDT m=+6450.256418038 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec
[NFO] 2017-09-15 11:22:53.677750821 -0400 EDT m=+6460.256814527 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec

etc...

majestrate commented 7 years ago

Configurable on runtime or I can change that entry's log level to debug On Fri, Sep 15, 2017 at 04:07:35PM +0000, l-n-s wrote:

Can it be optional?

[NFO] 2017-09-15 11:22:23.676528458 -0400 EDT m=+6430.255593282 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec [NFO] 2017-09-15 11:22:33.676941919 -0400 EDT m=+6440.256006044 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec [NFO] 2017-09-15 11:22:43.677353704 -0400 EDT m=+6450.256418038 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec [NFO] 2017-09-15 11:22:53.677750821 -0400 EDT m=+6460.256814527 Global Speed: Upload: 0.00B/sec Download: 0.00B/sec

etc...

— You are receiving this because you are subscribed to this thread. Reply to this email directly, [1]view it on GitHub, or [2]mute the thread.

Reverse link: [3]unknown

References

Visible links

  1. https://github.com/majestrate/XD/issues/23
  2. https://github.com/notifications/unsubscribe-auth/AAefxRBYMok7FDeA0cd5SuzyK1PlsJ_3ks5siqC9gaJpZM4PZMVu
  3. https://github.com/majestrate/XD/issues/23
l-n-s commented 7 years ago

Probably better to make it configurable, like if user needs that info in logs - he turns it on.

r4sas commented 7 years ago

I think better to make it configurable online - via rpc. Like showstats yes/no.

majestrate commented 7 years ago

It is already calculated by the current rpc tool.

On Sat, Sep 16, 2017 at 04:38:33PM +0000, R4SAS wrote:

I think better to make it configurable online - via rpc. Like showstats yes/no.

— You are receiving this because you commented. Reply to this email directly, [1]view it on GitHub, or [2]mute the thread.

Reverse link: [3]unknown

References

Visible links

  1. https://github.com/majestrate/XD/issues/23#issuecomment-329979634
  2. https://github.com/notifications/unsubscribe-auth/AAefxb37x8OQx2MtDMWzgBT5nW1RYOY6ks5si_mIgaJpZM4PZMVu
  3. https://github.com/majestrate/XD/issues/23#issuecomment-329979634