Skip to content

Add VBoxAuthSimple#15

Open
xqus wants to merge 2 commits into
phpvirtualbox:developfrom
xqus:xqus-vboxauthsimple
Open

Add VBoxAuthSimple#15
xqus wants to merge 2 commits into
phpvirtualbox:developfrom
xqus:xqus-vboxauthsimple

Conversation

@xqus

@xqus xqus commented Mar 28, 2016

Copy link
Copy Markdown

Adds VBoxAuthSimple so we don't have to store username and password in plain text.

To set up Virtualbox:
Configure VurtualBox web server to use VBoxAuthSimple.
$ VBoxManage setproperty websrvauthlibrary "VBoxAuthSimple"

Create a password hash.
$ VBoxManage internalcommands passwordhash "secret"

Add a user to the VirtualBox config. Username: username, with password: secret.
$ VBoxManage setextradata global "VBoxAuthSimple/users/username" / 2bb80d537b1da3e38bd30361aa855686bde0eacd7162fef6a25fe97bf527a25b`

Set VBoxAuthSimple as authentication library
var $authLib = 'VBoxAuthSimple';

h6w added a commit that referenced this pull request May 8, 2017
@h6w

h6w commented Oct 26, 2017

Copy link
Copy Markdown
Contributor

I completely agree with this PR, however we're going to need to be careful as far as version management is concerned. We'll need to document the change and the upgrade path for people using the current versions, so we may want to do this at a point release point.

@Kunzol

Kunzol commented Oct 26, 2017

Copy link
Copy Markdown

It seems this is the same as the "Builtin" authentication.

@h6w

h6w commented Oct 26, 2017

Copy link
Copy Markdown
Contributor

Yes, but it's different from what phpvirtualbox currently does, so some user migration will be required. No?

@Kunzol

Kunzol commented Oct 26, 2017

Copy link
Copy Markdown

I don't use it, but as far as I can see from the code, the username and password are stored in the extradata section of virtualbox.

If there is a difference, you have to explain it a bit more detailed.

@tom077

tom077 commented Jan 4, 2018

Copy link
Copy Markdown
Contributor

I have just tested this new authentication module.

It is totally different from the 'Builtin' authentication module which store username and password hash in virtualbox's extradata.

Instead of setting var $username and var $password in config.php, you can directly log in as the user running Virtualbox webserver, so that you don't have to store username and password in plain text in config.php.

Of course, this module is only relevant for a single user configuration.

However, the "vboxmanage" commands stated by @xqus are useless. The provided code doesn't look at Virtualbox "extradatas", it only checks that the given username and password are accepted by vboxwebsrv.

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.

4 participants