Add full file path to error logs that contain a stack about warrant HOT 4 OPEN

Hi @akajla09 I would like to contribute on this.
My proposed solution would be to change log.Logger = log.Output(zerolog.ConsoleWriter{Out: os.Stderr})
in config.go to log.With().Caller().Logger().Output(zerolog.ConsoleWriter{Out: os.Stderr}).
Let me know what you think about this.
PS: Thanks

from warrant.

Hi @AKARSHITJOSHI - thanks for offering to help. I haven't looked into this deeply so can't say for certain if this change will work. Have you tried it locally and tested it out? If you have a sample log output from the change, we can look at it to see if it contains the full filepath.

from warrant.

Please have a look at the following image

Message :
{"level":"fatal","error":"While parsing config: yaml: invalid trailing UTF-8 octet","time":"2023-04-09T11:05:49+05:30","caller":"/Users/personal/documents/warrant/pkg/config/config.go:88","message":"Error while reading warrant.yaml. Shutting down."}

The caller part is where the entire file path is shown . I tested the mentioned change in config.go

from warrant.

Can you submit a PR for this change? Looks like it does add the caller to error messages which is nice. However, I'm curious if it'll work for errors that are wrapped via Errors.wrap() (I can check that from your PR)

from warrant.