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

Issue 516: Bump ws library and test VTX #576

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

crhultay
Copy link
Contributor

#516

Bumped to 8.18.0 which is the latest

NuSkooler#516

Bumped to 8.18.0 which is the latest
@crhultay
Copy link
Contributor Author

Tested directly through http
image

Instructions in https://nuskooler.github.io/enigma-bbs/servers/loginservers/websocket.html, and this wants to be simplified into a bash script utility to make it easier for new SysOps.

Setup and tested via VTXClient per instructions in https://nuskooler.github.io/enigma-bbs/servers/loginservers/websocket.html, and this wants to be simplified into a bash script utility to make it easier for new SysOps.

I would see in my logs

[2024-11-23T00:48:11.768Z]  INFO: ENiGMA½ BBS/14182 on bbs: Client connected on node 1 (WebSocket/undefined) (nodeId=1, sessionId=W7UdzQD8g5Y, remoteAddress=::ffff:192.168.2.48, friendlyRemoteAddress=192.168.2.48, serverName=WebSocket, isSecure=false)

And from HTTP Server provided VTX Client, I see:
image

Telnetted into 192.168.2.12:8810 from another machine, and I get:
image

I'm not sure what is missing, but no errors are being logged by the server and the connection simply times out from the VTXClient whether I'm using ws 7.4.3 or ws 8.18.0.

#576 was raised, however I cannot affirm that VTX works or not.

@crhultay
Copy link
Contributor Author

Got it working. The example vtxdata.json was missing an important parameter.

@crhultay
Copy link
Contributor Author

Fixed, and tested on ws 8.18.0
image

@NuSkooler NuSkooler merged commit c2973fc into NuSkooler:master Nov 25, 2024
2 checks passed
@crhultay crhultay deleted the patch-3 branch November 25, 2024 01:48
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

Successfully merging this pull request may close these issues.

2 participants