[Feature Request] Add option to change express middleware log function

Is your feature request related to a problem? Please describe.

Currently, the Express logging middleware will output all logs as info, however this isn’t always wanted.

Describe the solution you’d like

Add a way to specify which log function to use. A third parameter in third place would be a possibility that avoids breaking changes; adding it as second parameter would be nicer though (no param behind a long and optional object).

Describe alternatives you’ve considered

Sure, I could write my own middleware. But what’s the point in shipping it with the package then ^^

[Olian04]

I agree that this might as well be a feature.

As I see it, there are 4 distinct ways of implementing this:

1. Change the first argument of the middleware from being the entire host obj, to just being the log method. app.use(expressMiddleware(console.info)) Pro: Clean, No API bloat Con: Breaking change
2. Add another argument to the function signature. app.use(expressMiddleware(console, {}, console.info)) Pro: None breaking Con: API bloat, First argument wont be used at all
3. Reuse config object, and add another property to it. app.use(expressMiddleware(console, { meta: { method: 'info' } }) Pro: None breaking Con: Might cause confusion since the config object has so far been used to configure WHAT is logged, not HOW it’s logged.
4. Overload the first argument of the middleware. Allowing the user to either provide the entire host obj, or provide just the log method. app.use(expressMiddleware(console)) and app.use(expressMiddleware(console.info)) would both be valid Pro: Clean, Low API bloat, Allows for a clean deprecation of “providing a host obj” between major versions Con: Implementation bloat

I’m leaning towards nr:2 for the time being, but then include nr:1 in the next major release. I’m leaning towards nr:4, since it’s a merge between nr:1 & nr:2

Relevant code:

https://github.com/Olian04/better-logging/blob/576a9f7a27cc68665b74ba3f72818462c7f23c02/src/express/index.ts#L7-L77

[Olian04]

This has been implemented in the latest release.

I decided to go with the overload approach. Overloading the first argument of the middleware. Allowing the user to either provide the entire host object, or provide just the specific log method. This means that both app.use(expressMiddleware(console)) and app.use(expressMiddleware(console.info)) would be valid. However bear in mind that app.use(expressMiddleware((...args) => console.info(...args))) would not be valid, this was done in order to not break #66