search

ovh / node-ovh

Node.js wrapper for the OVH APIs
http://ovh.github.io/node-ovh
Other
129 stars 27 forks source link

error on npm update #7

Closed miton18 closed 8 years ago

miton18 commented 8 years ago

Hi, I've just updated my project and now, something not work, on ovh module, the error is : 

TypeError : Proxy.create is not a function
refer to : ovh.js on line 226

Someone ever had this error?

VincentCasse commented 8 years ago

Can you send a sample of the code you use to have this error?

miton18 commented 8 years ago

Yes, this is Coffee

var conf = configService.getConfig();
if( conf.appKey == null ) {
    $window.location.replace( '#/connect/true' );
    return;
}
this.ovh = (require('ovh'))(conf);

and conf object:

{
    "endpoint": "kimsufi-eu",
    "appKey": "xxxx",
    "appSecret": "xxx"
}

And if it can help you : 

$ node -v
v5.8.0
yadutaf commented 8 years ago

Hi @miton18,

Did you try with node 4.4.0 LTS ? The code has not changed in a while. It seems unlikely that a regression has been introduced recently. My guess would be that the code may not be compatible with newer versions of Node. But i may be wrong.

Can you try as much as possible to use pure JS example ? This will help other people finding solutions through existing issues without learning dialects. Even though Coffee is pretty popular :)

miton18 commented 8 years ago

OK, in reality I use Eletron atom to run my app, and yes I have downgrade Electron-prebuilt and it works, I think there are some changes in electron's node engine Any upgrade of node-ovh will be plan?

yadutaf commented 8 years ago

Can you open a dedicated issue mentioning the upgrade with a plain js code sample which is broken in the upgrade ? Thanks for reporting.