sails-rest icon indicating copy to clipboard operation
sails-rest copied to clipboard

Upgrade restify to >=3.0.0

Open joshuajabbour opened this issue 9 years ago • 4 comments

Currently, sails-rest installs [email protected]. And with node 0.12 or iojs, I get these errors:

{ [Error: Cannot find module './build/Release/DTraceProviderBindings'] code: 'MODULE_NOT_FOUND' }
{ [Error: Cannot find module './build/default/DTraceProviderBindings'] code: 'MODULE_NOT_FOUND' }
{ [Error: Cannot find module './build/Debug/DTraceProviderBindings'] code: 'MODULE_NOT_FOUND' }

From what I've been able to figure out, this is fixed by using dtrace-provider >=0.4.0, which [email protected] satisfies.

joshuajabbour avatar May 25 '15 00:05 joshuajabbour

Were you able to run sails-rest with restify 3.0.0 ? I'm happy to accept a PR that resolves this

zohararad avatar May 25 '15 16:05 zohararad

@joshuajabbour please take a look at v0.1.0-waterline-v0.10 . It's a complete re-write of the adapter (doesn't use restify anymore). I plan to let it sit in that branch for a while before releasing, but it might be a good way to work around the problem you described above.

zohararad avatar Jun 06 '15 19:06 zohararad

@zohararad its great to see oncoming official superagent version! I made dirty port before to use in isomorphic fluxible application. Let me know if I can help somehow.

ertrzyiks avatar Jun 19 '15 12:06 ertrzyiks

@ertrzyiks glad for the positive comments. I'd love to get some feedback on

  1. Code quality if you're happy looking through it
  2. Usefulness in a real application. Most bugs come from real-world usage and I may have missed some things

zohararad avatar Jun 19 '15 13:06 zohararad