-
Notifications
You must be signed in to change notification settings - Fork 669
Add VNC video display including password #1004
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
Conversation
Probably should move the display of the password, or perhaps just show the filename ?
Some viewers accept a passwordfile, rather than prompting for the (clear-text) password. |
Oops, forgot to seed the random number generator so the generated "password" is always the same. |
Should show a real URL, so that it is clickable in the console (maybe include the VNC password too?) The current output works in vncviewer:
|
Note that the default user doesn't have any password, so still needs
|
So this works great to get the console, but how do you log in? |
Sorry, was confused; I just realized that this is exactly the same as the regular console 😄 |
It also depends on the example, and if you add a login manager you could also add more login methods ? The VNC "password" is just to protect the display as such, it doesn't change anything about authentication Otherwise it is no different from any other qemu |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
507e2bc
to
03e9367
Compare
Now the default vnc display will pick the first available one, from This means that all the user needs to to do is to change the display type from "none" to "vnc", and get the password. It can either be seen in the log, or read from the instance file. The log will also show the display number, if it wasn't fixed. Maybe the display needs a separate file, next to the password. Then you could just read the |
SGTM. Btw, I guess the VNC feature should be marked as an experimental for now, at least until we can have integration tests (how?)? |
CI failing |
I couldn't really find any integration tests for the other display settings either, except for the default of |
Just because I said that, I actually managed to break "default" in the latest re-design (when moving to Video.Display)
oops. |
Broken by the driver introduction |
Added some new functions to the BaseDriver, need to verify that it is QEMU before allowing VNC... ChangeDisplayPassword(_ context.Context, password string) error
GetDisplayConnection(_ context.Context) (string, error) "port" seemed a little specific, so went with a more generic "connection" (even if only vnc uses it) |
Needs rebase |
For some versions of QEMU, the only display option is VNC. |
The other display options open a window always, while the vnc is more "on demand" by using a separate vnc viewer. Add localhost and password support for some minimal security. The password is generated, and is stored as an instance file. Signed-off-by: Anders F Björklund <[email protected]>
It makes sure to use crypto/rand instead of math/rand. Also includes all the characters, and not just digits. Signed-off-by: Anders F Björklund <[email protected]>
Signed-off-by: Anders F Björklund <[email protected]>
Signed-off-by: Anders F Björklund <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
The other display options open a window always, while the
vnc is more "on demand" by using a separate vnc viewer.
Add localhost and password support for some minimal security.
The password is generated, and is stored as an instance file.
Closes #1003