Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[fix] Was failing test 'winston.accept' with winston >= v0.5.6 (and possibly earlier). #2

Merged
merged 1 commit into from Nov 27, 2011

Conversation

Projects
None yet
2 participants
Contributor

forddg commented Nov 26, 2011

node accept/winston.accept.js

node.js:134
throw e; // process.nextTick error, or 'error' event on first tick
^
TypeError: Object [object Object] has no method 'on'
at [object Object].add (/path/to/node-logentries/node_modules/winston/lib/winston/logger.js:284:12)
at Object.add (/path/to/node-logentries/node_modules/winston/lib/winston.js:81:34)
at Logger.winston (/path/to/node-logentries/lib/logentries.js:268:13)
at Object. (/path/to/node-logentries/accept/winston.accept.js:10:5)
at Module._compile (module.js:402:26)
at Object..js (module.js:408:10)
at Module.load (module.js:334:31)
at Function._load (module.js:293:12)
at Array. (module.js:421:10)
at EventEmitter._tickCallback (node.js:126:26)

All tests pass using winston v0.5.6/7 and node v0.4.10.

vilda added a commit that referenced this pull request Nov 27, 2011

Merge pull request #2 from forddg/master
[fix] Was failing test 'winston.accept' with winston >= v0.5.6 (and possibly earlier).

@vilda vilda merged commit 702dad4 into rapid7:master Nov 27, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment