adonisjs / logger

AdonisJS Logger built on top of pino
https://docs.adonisjs.com/guides/logger
MIT License
23 stars 9 forks source link

sprintf doesn't work #39

Closed pdipax closed 4 years ago

pdipax commented 4 years ago

Hi, given that I use the default logger version in adonis, I find that it does not recognize the sprintf functionality.

Es: Logger.info("request is %j", { url: request.url(), controller: 'EventController' })

returns Mar 17 11:20:22.257am info request is %j without translate %j

kauanmocelin commented 4 years ago

Occurs same with me, but only when transport is type 'file'.

thetutlage commented 4 years ago

Sorry, A bit late to the party. But can you share the version of logger you are using. I just added a test case and seems to be working fine.

thetutlage commented 4 years ago

Closing since no response from the issue reporter and not actionable as well

adeel94mirza commented 4 years ago

@thetutlage I am facing this issue.

Console info: serving app on http://127.0.0.1:3333 info: User Logged In: you@you.com

File {"level":"info","message":"serving app on http://%s:%s"} {"level":"info","message":"User Logged In: %s"}

I can provide more info if needed.