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

Add support for VNC specific passwords #24

Open
jyundt opened this issue Dec 6, 2017 · 2 comments
Open

Add support for VNC specific passwords #24

jyundt opened this issue Dec 6, 2017 · 2 comments

Comments

@jyundt
Copy link

jyundt commented Dec 6, 2017

Any thoughts on adding support for a separate VNC specific password?

I ran into an issue where my AMT password did not meet the RFBPassword / VNC password requirements because it was > 8 characters.

Maybe prompt the user when enabling KVM/VNC and update hosts.cfg?

Let me know if this sounds reasonable and I'll submit a PR to add this functionality.

@sdague
Copy link
Owner

sdague commented Dec 6, 2017

A separate password for VNC would be great.

How about add a flag to the enable vnc sub command that will trigger the prompt, so it's optional.

@jyundt
Copy link
Author

jyundt commented Dec 6, 2017

A separate password for VNC would be great.

Cool, I'll take a stab at this.

How about add a flag to the enable vnc sub command that will trigger the prompt, so it's optional.

Parsing an additional flag for a possible VNC password might get a little bit awkward because the "op" (on|off|reboot|pxeboot|status|vnc|vncstatus) is not a sub-command.

Do you mind if I get a separate PR going to modify the argparse config to include sub-commands?

Also, thanks for writing amt! I just got a vPRO enabled NUC and this utility is a life saver.

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

2 participants