Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

websockify.js : Server disconnected (code: 1006) #3

Open
jude90 opened this issue Oct 25, 2017 · 6 comments
Open

websockify.js : Server disconnected (code: 1006) #3

jude90 opened this issue Oct 25, 2017 · 6 comments
Assignees

Comments

@jude90
Copy link

jude90 commented Oct 25, 2017

node : v.6.11.5
[email protected]
#websockify crashed when I conneccted at vnc.html
trace back stack :

/home/vagrant/node_modules/websockify/websockify.js:129
        callback(true, 'binary');
        ^

TypeError: callback is not a function
    at Object.selectProtocol [as handleProtocols] (/home/vagrant/node_modules/websockify/websockify.js:129:9)
    at WebSocketServer.handleUpgrade (/home/vagrant/node_modules/ws/lib/WebSocketServer.js:169:31)
    at Server.WebSocketServer._ultron.on (/home/vagrant/node_modules/ws/lib/WebSocketServer.js:87:14)
    at emitThree (events.js:116:13)
    at Server.emit (events.js:194:7)
    at onParserExecuteCommon (_http_server.js:409:14)
    at Socket.socketOnData (_http_server.js:371:5)
    at emitOne (events.js:101:20)
    at Socket.emit (events.js:188:7)
    at readableAddChunk (_stream_readable.js:176:18)

updated:
full dependency as follows
└─┬ [email protected]
├─┬ [email protected]
│ ├── [email protected]
│ └── [email protected]
├── [email protected]
└─┬ [email protected]
├── [email protected]
├── [email protected]
└── [email protected]

@CendioOssman
Copy link
Member

You have some incompatibility between websockify and the version of WebSocketServer you are using. I would suggest upgrading to the latest version of websockify.js.

@DirectXMan12, perhaps time to push something updated to npm?

@evantobin
Copy link

@DirectXMan12 Any update on upgrading the version in NPM?

@mliudev
Copy link

mliudev commented May 22, 2018

Getting a similar error with the latest node on Windows Server 2008 R2:

PS C:\> node --version
v10.1.0
PS C:\> npm --global ls
`-- websockify@0.7.1
  +-- optimist@0.6.1
  | +-- minimist@0.0.10
  | `-- wordwrap@0.0.3
  +-- policyfile@0.0.6
  `-- ws@5.2.0
    `-- async-limiter@1.0.0

I'm getting the following error:

WebSocket settings:
    - proxying from :7777 to localhost:5900
    - Running in encrypted HTTPS (wss://) mode using: foo.pem, foo.pem
C:\Users\Administrator\AppData\Roaming\npm\node_modules\websockify\websockify.js:129
        callback(true, 'binary');
        ^

TypeError: callback is not a function
    at Object.selectProtocol [as handleProtocols] (C:\Users\Administrator\AppData\Roaming\npm\node_modules\websockify\we
bsockify.js:129:9)
    at WebSocketServer.completeUpgrade (C:\Users\Administrator\AppData\Roaming\npm\node_modules\websockify\node_modules\
ws\lib\websocket-server.js:260:33)
    at WebSocketServer.handleUpgrade (C:\Users\Administrator\AppData\Roaming\npm\node_modules\websockify\node_modules\ws
\lib\websocket-server.js:220:10)
    at Server.upgrade (C:\Users\Administrator\AppData\Roaming\npm\node_modules\websockify\node_modules\ws\lib\websocket-
server.js:78:16)
    at Server.emit (events.js:182:13)
    at onParserExecuteCommon (_http_server.js:535:14)
    at onParserExecute (_http_server.js:482:3)

Does anyone know of a working version of node and this version of websockify? At some point there was a working combination.

This issue seems to indicate that windows was working at some point in the beginning of this year.
novnc/websockify#67 (comment)

@mliudev
Copy link

mliudev commented May 22, 2018

As an update I was able to get this working with this version of websockify.js using node 10.1.0. I think I just need to package this up as a node module and install it globally.

@alex88
Copy link

alex88 commented Jan 3, 2019

The problem is the ws library version, instead of using >=0.4.27 use ~0.4.27 as dependency constraint in package.json and everything works.
Problem is that ws is now at version 6.1.2 so quite far from the 0.x this package is using

@CendioOssman
Copy link
Member

Ah, thanks. We'll have to update our dependencies then. Thanks for figuring it out.

@samhed samhed transferred this issue from novnc/websockify Jul 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants