Deprecation warning when using import

[rigwild]

When using import with esm using morgan, I get the following warning message :

morgan deprecated default format: use combined format node_modules\esm\esm.js:1:277757

This does not occur when using require.
Example : https://repl.it/repls/SeagreenGrippingHexagon

On Morgan v1.9.1.

[dougwilson]

Thanks for the report. This should only display if you're trying to get the deprecated morgan.default property, though. I'll need to dig in to see what is going on.

[rigwild]

import loads default by default. This default property needs to be renamed. There's probably a workaround for this but I can't help.

[dougwilson]

So it will be renamed eventually, which is why it is deprecated.

[dhowe]

Same problem here. Is there anyway to get rid of the distracting 'deprecated' warning without forking and changing the source code?

[dougwilson]

I haven't yet investigated what changes can be made to this module to fix the deprecation warning for the import syntax, but the warnings can always be suppressed from printing, though node command line switch, event listener, environment variable. You can read more here about how the deprecation system works if that helps at all for the time being: https://www.npmjs.com/package/depd

[dhowe]

perhaps remove the deprecation warning until you have time to investigate further?

[dougwilson]

The deprecation warning was added almost 5 years ago and there was no issue until this issue. Perhaps if this issue was opened closer to when it was added reverting would be considered, but whatever has changed since then isn't this module.

[dhowe]

My guess is that people are simply using the 'import' style more now.
The fix is just a comment (see this PR), but of course its your call...

[mbrowne]

Here is a fix that leaves the deprecation message in place: #207

Note that module.exports.default is still pointing to the default format function. I'm not sure if that might confuse any other tools that parse ES6 exports, but this at least prevents the deprecation warning from being erroneously shown when using esm.

[josemvcerqueira]

Quick fix is to pass "tiny" instead of dev to morgan.

[mbrowne]

@dougwilson I tried updating my app to use native ES modules now that they have landed in node (since node 13.2.0), but ran into some issues, so need to stick with esm for now. (It seems it will be a while before the ecosystem catches up with native ES modules to make adopting them a little more seamless.) So if you were to release a new major version of morgan that drops the deprecated default format (as you previously mentioned as a possibility), that would still be very helpful in the meantime.

[laurlas]

This will do the trick
import * as morgan from 'morgan'