search

squeeks / glossy

syslog parser and producer
https://npmjs.org/package/glossy
MIT License
96 stars 29 forks source link

some cisco routers produce a message of form <32>9032: 51w2d: ... message no hostname, #5

Closed fygrave closed 12 years ago

fygrave commented 12 years ago

...ssage

squeeks commented 12 years ago

Thank you very much for this, however would it be possible for you to include tests? Just look a look at test/parse.js and include some to cover whatever cases are for this.

fygrave commented 12 years ago

Sure. I'll add tests.

fygrave commented 12 years ago

Tests included. I included also some logs generated by buggy (i.e. null included) and non-english systems (chinese this case).

squeeks commented 12 years ago

I can't merge this in until the tests pass, at present your fork is broken.

fygrave commented 12 years ago

Apologize for not testing it on parse.js apriori. I moved my tests to the bottom of the array, so it runs properly now.