Project

General

Profile

Actions

Bug #31317

closed

VNC Console broken

Added by Christian Ruppert over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
VM management
Target version:
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Hi,

the VNC connection in 2.2.0 was broken because it was using "ws://" instead of "wss://" in my case (:websockets_encrypt: on).
It only showed "Disconnected." in the console tab. Nothing else/more.
Turning debug logging on only resulted in nonsense "error" messages:
2020-11-16T10:14:15 [D|app|46df8879] VNCProxy Error: WebSocket server settings:
46df8879 |
2020-11-16T10:14:15 [D|app|46df8879] VNCProxy Error: - Listen on :5929
46df8879 |
2020-11-16T10:14:15 [D|app|46df8879] VNCProxy Error: - Flash security policy server
46df8879 |
2020-11-16T10:14:15 [D|app|46df8879] VNCProxy Error: - SSL/TLS support
46df8879 |
2020-11-16T10:14:15 [D|app|46df8879] VNCProxy Error: - Backgrounding (daemon)
46df8879 |

So those errors, which weren't actual errors, didn't help.

This was likely introduced via #27319, especially https://github.com/theforeman/foreman/pull/6907/files#diff-acf85befb293c237348f2c37f605909896cb7240beca4c7d7ad7a62d5fe8ead5R20
tbrisker provided a possible fix via IRC:
idl0r: can you try changing https://github.com/theforeman/foreman/pull/6907/files#diff-acf85befb293c237348f2c37f605909896cb7240beca4c7d7ad7a62d5fe8ead5R20 to data-encrypt='<%= @encrypt %>' ?

Which we did and it worked afterwards.
Previously the JS error console of the browser logged "Failed when connecting: Error when opening socket (SecurityError: The operation is insecure.)"


Related issues 1 (0 open1 closed)

Related to Foreman - Feature #27319: VNC console for kubevirtClosedShira MaximovActions
Actions #1

Updated by Tomer Brisker over 3 years ago

Actions #2

Updated by Tomer Brisker over 3 years ago

  • Category set to VM management
  • Target version set to 2.2.2
Actions #3

Updated by The Foreman Bot over 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Shira Maximov
  • Pull request https://github.com/theforeman/foreman/pull/8138 added
Actions #4

Updated by The Foreman Bot over 3 years ago

  • Fixed in Releases 2.4.0 added
Actions #5

Updated by Tomer Brisker over 3 years ago

  • Fixed in Releases 2.2.2, 2.3.0 added
  • Fixed in Releases deleted (2.4.0)
Actions #6

Updated by Tomer Brisker over 3 years ago

  • Subject changed from VNC Console broken through #27319 to VNC Console broken
Actions #7

Updated by Shira Maximov over 3 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF