refactor: logging [BREAKING CHANGE]

chimurai commented 2 years ago

Description

feat(option): logger
removed(option): logProvider and logLevel
docs(logger): examples with console, winston, pino, bunyan, log4js
refactor: improved request/response logging

Motivation and Context

simplify internals with better logging and improved compatibility with external loggers.

fixes #123 fixes #194 fixes #351 fixes #747

How has this been tested?

docs
unit tests

Types of changes

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

Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

coveralls commented 2 years ago

Coverage increased (+0.06%) to 97.0% when pulling 05ef657a7decf5707e3650a928e14df305b74953 on refactor-logger into 33f9f5ae4711230ed8a7a2f03d7d4924cb3d9a15 on v3.

dandv commented 1 year ago

Looks like an old types.d.ts was still shipped with the 2.0.6 release, so the logger option generates a TypeScript error.

Can you please take a look?

chimurai commented 1 year ago

Did you try to remove "node_modules" and install your dependencies again?

Also, looks like your using v3 options within v2... V2 docs: https://github.com/chimurai/http-proxy-middleware/tree/v2.0.4#readme

chimurai / http-proxy-middleware