Skip to content

Fix Port configuration for novncd#352

Merged
retspen merged 1 commit intoretspen:masterfrom
shokinn:fix_novncd_port
Aug 18, 2020
Merged

Fix Port configuration for novncd#352
retspen merged 1 commit intoretspen:masterfrom
shokinn:fix_novncd_port

Conversation

@shokinn
Copy link
Copy Markdown
Contributor

@shokinn shokinn commented Aug 13, 2020

Revert accidentally change variable for novncd port configuration.

Change variable WS_PUBLIC_PORT back to WS_PORT

Reverte accidentally change variable for novncd port configuration.

Change variable `WS_PUBLIC_PORT` back to `WS_PORT`
@shokinn shokinn mentioned this pull request Aug 13, 2020
@retspen
Copy link
Copy Markdown
Owner

retspen commented Aug 17, 2020

Hello,

I'm not sure about that changes. WS_PUBLIC_PORT variable need for fronted side and use for redirect or URL open.

@lord-kyron
Copy link
Copy Markdown
Contributor

@retspen - this is not a change but a fix as in the last commit @catborise changed this in the wrong place and VNC stopped working for us. That is why @shokinn is reverting this back as we tested it and that was the problem.

@shokinn
Copy link
Copy Markdown
Contributor Author

shokinn commented Aug 18, 2020

Yupp.
Thanks @lord-kyron for clarifying!

@retspen See discussion here: #343 (comment)

@retspen
Copy link
Copy Markdown
Owner

retspen commented Aug 18, 2020

I understood. Thanks for help!

@retspen retspen merged commit a132470 into retspen:master Aug 18, 2020
@lord-kyron
Copy link
Copy Markdown
Contributor

@shokinn - you can join the official Webvirtcloud Discord channel here:
https://discord.gg/WVbkSW
There we can discuss and fix issues faster and easier on chat :)

@shokinn shokinn deleted the fix_novncd_port branch August 19, 2020 11:21
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.

3 participants