Fixes immediate WS disconnect error with NodeJS v6.3+

[tautologistics]

Fixes small error in handling error condition vs. having an actual native module to return:

TypeError: Cannot read property 'ref' of undefined
    at InjectorClient.<anonymous> (/usr/lib/node_modules/node-inspector/lib/InjectorClient.js:111:22)
    at /usr/lib/node_modules/node-inspector/lib/DebuggerClient.js:121:7
    at Object.value (/usr/lib/node_modules/node-inspector/lib/callback.js:23:20)
    at Debugger._processResponse (/usr/lib/node_modules/node-inspector/lib/debugger.js:95:21)
    at Protocol.execute (_debugger.js:121:14)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)
    at Socket.Readable.push (_stream_readable.js:134:10)
    at TCP.onread (net.js:543:20)

Reference issues:

• Throwing exception on simple use case #905
• Detached from the target, websocket_closed, cannot read property ref of undefined (NM[0] is undefined) #907
• not working with new version nodejs 6.4.0+ #928
• Remote debugging as been terminated: websocket closed #931

[samuelcastro]

Is this gonna be merged?

[MadOgre]

Not sure if the maintainers check this repo anymore. Maybe time for a fork.

[pospi]

I've manually applied this fix and I now get Error: No NativeModule in target scope in the debugger- no source files, still doesn't appear to work.

[samuelcastro]

I've just decided to use native inspector with: node --inspect index.js and it's working nicely. (https://nodejs.org/api/debugger.html#debugger_v8_inspector_integration_for_node_js)

[pospi]

heh, likewise...

On Fri, Oct 28, 2016 at 12:03 PM, Samuel Castro notifications@github.com
wrote:

I've just decided to use native inspector with: node --inspect index.js
and it's working nicely. (https://nodejs.org/api/
debugger.html#debugger_v8_inspector_integration_for_node_js)

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#932 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAatJKXtUUl-J3nrFGe5GjlEyrnBLLXgks5q4VfZgaJpZM4KMpi-
.

[grantgeorge]

@samuelcastro @pospi do you guys run into the issue where it opens a new window for the inspector whenever your server reloads? kind of a pain in the but using the --inspect flag for me

[zignd]

@samuelcastro thanks for the tip!

The --inspect flag on the Node.js runtime is a great alternative!

[hustxiaoc]

Fixed in node-inspector@1.0.0