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

add additional details to error logs #102

Open
jonathanstrong opened this issue Feb 24, 2021 · 1 comment
Open

add additional details to error logs #102

jonathanstrong opened this issue Feb 24, 2021 · 1 comment
Assignees
Labels
C-enhancement Category: Enhancement P-medium Priority: Medium

Comments

@jonathanstrong
Copy link

hey, just a user report from the wild for you to consider.

I was setting up alexandrie behind a reverse proxy and ran into a puzzling error.

it turned out to be this issue.

the alexandrie logs displayed this:

Feb 24 11:48:20.085 ERRO async-h1 error, version: 0.1.0                                                                

I tried but was unable to identify where in the source code that message is generated. It would have been better if there were additional details provided, to facilitate debugging the problem. thanks!

@Hirevo Hirevo added C-enhancement Category: Enhancement P-medium Priority: Medium labels Mar 21, 2021
@Hirevo
Copy link
Owner

Hirevo commented Mar 21, 2021

Hello,

I agree that log messages are currently too minimal.
One way to improve things could make better use to structural logging and maybe make log statements be done as NDJSON (newline-delimited JSON) in release mode, with an optional simpler pretty printer when built in debug mode.

I'll try to poke at it very soon, Thanks for filing this issue !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: Enhancement P-medium Priority: Medium
Projects
None yet
Development

No branches or pull requests

2 participants