search

cnodejs / nodeclub

:baby_chick:Nodeclub 是使用 Node.js 和 MongoDB 开发的社区系统
http://cnodejs.org/
MIT License
9.34k stars 3.12k forks source link

node库oneapm 1.2.8在node4.1.1导致程序启动报错 #561

Open nqdy666 opened 9 years ago

nqdy666 commented 9 years ago

我fork的cnodejs源码在自己电脑上部署。 在node0.12.0，程序运行正常，oneapm 也运行正常。 在使用node4.1.1，只要把var oneapm = require(‘oneapm’)注释掉，程序运行是正常的，一加上就导致程序启动爆错。 不知道cnodejs社区在测试node4.1.1版本时，有没有遇到一样的问题。

albin3 commented 9 years ago

这应该是oneapm方面代码的问题吧。。

alsotang commented 9 years ago

@wyvernnot 看一看？

alsotang commented 9 years ago

我看 1.2.8 已经是 oneapm 最新的了。等待 oneapm 的同学回复中。

2015-09-29 17:25 GMT+08:00 Albin. Zeng notifications@github.com:

这应该是oneapm方面代码的问题吧。。

— Reply to this email directly or view it on GitHub https://github.com/cnodejs/nodeclub/issues/561#issuecomment-144004483.

longtian commented 9 years ago

跟进中

alsotang commented 9 years ago

@wyvernnot 有消息了吗

longtian commented 9 years ago

已经在 oneapm@1.2.9 里修复，这个问题是由于 binded function 在 Node.JS 不同版本下 name 不一致导致的。

举例说明:

Math.abs.bind(Math).name 的返回值在 node4.0 之前为空，在 node4.0 里是 bound ，但是在 node4.１及后来的版本里是 bound abs

相关讨论:

https://github.com/nodejs/node/issues/2754

目测　#560 也是同样的原因导致的

alsotang commented 9 years ago

https://github.com/cnodejs/nodeclub/commit/3c742e735dec48a8c1951faec96892c15211c228

我把 oneapm 升级到 1.2.9 了。

alsotang commented 9 years ago

话说，oneapm 的 nodejs 探针是开源的吗？