Skip to content
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

Redis transport returns 'Invalid transport, must be an object with a log method' #1252

Closed
ADumaine opened this issue Mar 30, 2018 · 2 comments

Comments

@ADumaine
Copy link

Using

require('winston-redis').Redis;
winston.add(winston.transports.Redis);

I have tried passing options and also creating a redis client first which I think should be recognized by winston-redis.

winston-redis also does not seem to be compatible. There is no longer any common.log in common.js called by winston-redis.
https://github.com/winstonjs/winston-redis/blob/master/lib/winston-redis.js#L101

Is there any way to use Redis with 3.x?

@DABH
Copy link
Contributor

DABH commented Apr 1, 2018

Thanks for asking! I'll close this in favor of your other issue, winstonjs/winston-redis#28, but the short answer is that we haven't gone through and verified compatibility of all the transports -- we're still working on core 3.x dev as a priority. We would welcome any PRs to transports to make compatible again though. Might not be too hard if you want to take a look, e.g. common.log is maybe now something like winston.log or winston.default.log ? (would have to look more...)

@DABH DABH closed this as completed Apr 1, 2018
@binarymist
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants