add logger, console->logger migration - Githubissues

camicroscope / Caracal

Conslidated Backend, Auth, and Security Services for caMicroscope

GNU General Public License v3.0

15 stars 95 forks source link

add logger, console->logger migration #79

Closed YashKumarVerma closed 3 years ago

YashKumarVerma commented 3 years ago

Description

adds Winston as a dependency for logger
adds transports to write logs to file
adds separate transports for errors and Infos
migrates existing codebase from console -> logger

Motivation and Context

as discussed in #76
Fixes #76

How Has This Been Tested?

linting tests
existing tests do not fail.

Screenshots (if appropriate):

N/A
Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

YashKumarVerma commented 3 years ago

Closing in support of further development.