metatech-university / NodeJS-Fastify

Fastify + NodeJS application server
MIT License
21 stars 17 forks source link

Write logs to the file #7

Closed RohovAlex closed 1 year ago

RohovAlex commented 1 year ago

Writing logs to the file was added. Closes: https://github.com/metatech-university/NodeJS-Fastify/issues/4

tshemsedinov commented 1 year ago

It will be good to add following tag in commit description

Closes: https://github.com/metatech-university/NodeJS-Fastify/issues/4

tshemsedinov commented 1 year ago

Hint: I'd advise to use git rebase instead of merge. That's about https://github.com/metatech-university/NodeJS-Fastify/pull/7/commits/f5cf16ddec5b75fc7bfbad22b3b2a014dd044e29

JekRock commented 1 year ago

As an inspiration:

But I personally like the approach suggested by pino - using external tools for log rotation, like logrotate.

RohovAlex commented 1 year ago

@nechaido Done, please review.

RohovAlex commented 1 year ago

@tshemsedinov @nechaido please review