Log output to filesystem

ChainCashLabs / chaincash-rs

ChainCash Server: Free Banking for Everyone!

Creative Commons Zero v1.0 Universal

6 stars 4 forks source link

Log output to filesystem #37

Closed SethDusek closed 9 months ago

SethDusek commented 9 months ago

For #5

Some questions:

Is rotating log files daily fine or should I adjust the rotation interval?
Writing logs to ./chaincash.log.d/chaincash.log*. I was looking for where the standard place to put log files on Linux for user programs is, and there doesn't seem to be one (besides /var/log which requires special privileges to create a log directory). So I assume putting this in the current directory is fine.

ross-weir commented 9 months ago

Is rotating log files daily fine or should I adjust the rotation interval?

yeah that's fine

Writing logs to ./chaincash.log.d/chaincash.log*. I was looking for where the standard place to put log files on Linux for user programs is, and there doesn't seem to be one (besides /var/log which requires special privileges to create a log directory). So I assume putting this in the current directory is fine.

Lets add directories-rs dependency and put logs in the dirs specified here: https://github.com/dirs-dev/directories-rs/issues/81

SethDusek commented 9 months ago

Lets add directories-rs dependency and put logs in the dirs specified here: dirs-dev/directories-rs#81

Done! Also watching the github issue so if (and when) this is built-in to directories-rs I can replace our implementation with that

ross-weir commented 9 months ago

Lets add directories-rs dependency and put logs in the dirs specified here: dirs-dev/directories-rs#81

Done! Also watching the github issue so if (and when) this is built-in to directories-rs I can replace our implementation with that

Amazing, thank you!

Just one more small thing and will merge.

Whats your discord/telegram? I'll get kushti to contact you for bounty payment

SethDusek commented 9 months ago

Lets add directories-rs dependency and put logs in the dirs specified here: dirs-dev/directories-rs#81

Done! Also watching the github issue so if (and when) this is built-in to directories-rs I can replace our implementation with that

Amazing, thank you!

Just one more small thing and will merge.

Whats your discord/telegram? I'll get kushti to contact you for bounty payment

My Discord username is dusek_. Thanks!

kushti commented 9 months ago

@SethDusek please provide your Ergo address (ordinary or stealth) over Discord (sent you DM there) or email kushti@protonmail.ch