search

Zokalyx / theconfluence

1 stars 2 forks source link

Logging is too cluttered #3

Closed Zokalyx closed 2 years ago

Zokalyx commented 2 years ago

Seems like Heroku already provides some of the same logging format as the one that's in the code image

_Originally posted by @Zokalyx in https://github.com/Zokalyx/theconfluence/pull/1#discussion_r795077571_

Thejusko commented 2 years ago

You can change the output log level in the logging config at the start of the file. WARN as example will only output LOGGER.warn and LOGGER.error

Which data does Heroku provides? ( The Screenshot doesn't contain anything )

Zokalyx commented 2 years ago

I still want to see info level log, but I want each print to fit in its own line.

Yeah my bad, I automatically assumed that all of the black text was part of the logging, but apparently not. I was talking about 2022-0129T17:26:53.583408+00:00 app[worker.1]. It doesn't look like this can be modified :(

Zokalyx commented 2 years ago

As a "temporary" measure, I will remove the timestamps from the new logger. I won't change this repo, just the Heroku one. (Maybe I should sync the changes?)

EDIT: I'm also modifying the lengths of each "part" of the logging format and adding a newline right before the content of the log. New logs look like this:

image