aftership / deprecated-koa-newrelic Goto Github PK

Still see /* in NewRelic.

Problem is also mentioned in StackOverflow:
http://stackoverflow.com/questions/38886721/how-to-configure-newrelic-on-node-koa-app

Hi all,

i have tried to use koa-newrelic but still get /* for all transactions.
currently i am using

• koa
• koa-router

i configure koa-newrelic from the example of this repo.
are there more configuration?

Hi, I found that the function newrelic.setTransactionName() is put after the yield next().
https://github.com/AfterShip/koa-newrelic/blob/2.0.0/index.js#L156-L162

However, in Koa2, the newrelic log this error:

{"v":0,"level":40,"name":"newrelic","hostname":"Adams-iMac.local","pid":26721,"time":"2017-07-11T05:11:51.042Z","msg":"No transaction found when setting name to 'GET /index'.","component":"api"}

Newrelic module throws this error at these lines because the transaction not found.
https://github.com/newrelic/node-newrelic/blob/master/api.js#L66-L69

Do you have any ideas about this?

npm test is showing 5 failing tests.

Are the tests simply out of date? I can help w/ travis integration if you are interested.

Also, is there any plan to add support for the Koa 2 alpha?

Is it possible to somehow configure which errors should be reported? For example, I'd like to skip 4xx errors from reporting. Also would be great to include additional headers/request params to error parameters. Thanks.