add ratchetio for error handling #14

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

brianr commented Nov 10, 2012

Hey guys --

I created a Ratchet.io project for this and added Gerad to it (you should've received an email invite). Feel free to remove me from the project after you accept.

I left the Airbrake code in so you can compare the difference if you want :)

-brian

Owner

gerad commented Nov 11, 2012

dude you rock, thanks!

Owner

gerad commented Nov 11, 2012

didn't merge cleanly, but i merged it in manually

gerad closed this Nov 11, 2012

Owner

gerad commented Nov 12, 2012

TypeError: Cannot call method 'split' of undefined
at Object.exports.parseStack (/Users/gerad/Development/nko/node_modules/ratchetio/lib/parser.js:25:21)
at Object.exports.parseException (/Users/gerad/Development/nko/node_modules/ratchetio/lib/parser.js:8:18)
at Object.exports.handleError (/Users/gerad/Development/nko/node_modules/ratchetio/lib/notifier.js:75:17)
at Object.exports.errorHandler as handle
at next (/Users/gerad/Development/nko/node_modules/express/node_modules/connect/lib/http.js:199:17)
at pass (/Users/gerad/Development/nko/node_modules/express/lib/router/index.js:219:24)
at nextRoute (/Users/gerad/Development/nko/node_modules/express/lib/router/index.js:209:7)
at callbacks (/Users/gerad/Development/nko/node_modules/express/lib/router/index.js:274:11)
at callbacks (/Users/gerad/Development/nko/node_modules/express/lib/router/index.js:269:37)
at param (/Users/gerad/Development/nko/node_modules/express/lib/router/index.js:239:11)

Owner

gerad commented Nov 12, 2012

5179f42 was the fix

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