Skip to content

Conversation

@tarun-kavipurapu
Copy link
Contributor

Under Logger Folder
Created a logger for Applications Errors with Winston and for logging http call using Morgon

Since the output format and the method was not mentioned used custom format and displayed the output to the Console

Happy to make any changes if further infromation is provided by the maintainers

Demo image which shows both application error and http log
image

tarun-kavipurapu and others added 5 commits February 7, 2024 22:46
- add comprehensive winston logger options
- remove status code based skip key in morgan
- fix typo
- fix logger file structure
Copy link
Collaborator

@wajeshubham wajeshubham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please take a pull of the changes to your local branch and resolve the reviewed changes. I have refactored file structure and morgan and winston logger configurations. Make sure you test the logging thoroughly before requesting a review.

Thanks!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use yarn as a package manager and not npm. Delete the package-lock.json and make sure yarn.lock only has newly installed dependency changes

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove package-lock.json and reinstall packages using yarn

@tarun-kavipurapu
Copy link
Contributor Author

Successfully Implemented the mentioned changes
Application Error log and http log
Screenshot from 2024-02-08 10-20-30

apologize for previous mistakes as this was my first time contributing

Copy link
Collaborator

@wajeshubham wajeshubham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@wajeshubham wajeshubham merged commit f6e8b74 into hiteshchoudhary:main Feb 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants