-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TypeError: transport is not a function #42
Comments
@brandonros can you identify if there is a different version of |
There is not. I can 'ncu -a' and 'npm update' on both. On Friday, February 26, 2016, Jarrett Cruger [email protected]
|
@brandonros try |
|
It has something to do with me being a noob when it comes to module.exports and its behavior/caching, I think. |
@brandonros try using logger = require('morgan') along with winston, I am facing same problem if I remove morgan
|
@brandonros did you manage to solve this? Seeing the same issue. :/ |
...found it out by myself eventually -- reason was in two different versions of
Due to that and some require magic I don't quite understand, transport set by So in my case the "fix" was to explicitly require latest winston at the root of the package, which would then re-shuffle winston deps (storing local version in prompt rather than in root). Hope this helps. |
Having the same problem as brandonros. When I place the code at index.js, everything runs, but when I place it in a module and require the module in index.js, I get the same error |
@LucasZanella I am getting the same issue as you, did you ever find a fix? |
Anybody ever solve this? |
I solved this by removing 'winston' and 'winston-loggly-bulk' Before I was installing winston-loggly-bulk first then winston. Not sure if it is the issue. |
log.js
Stack
I can't reproduce this on my development machine, but can reproduce it on my production servers.
The text was updated successfully, but these errors were encountered: