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

Provide a database password for the LAMPApp plugin #65

Open
quicksketch opened this issue Apr 30, 2016 · 3 comments
Open

Provide a database password for the LAMPApp plugin #65

quicksketch opened this issue Apr 30, 2016 · 3 comments
Assignees

Comments

@quicksketch
Copy link

I'm trying out Probo CI today for the first time. I have to say, it's a relief to have a CI platform that actually includes a web server! And it works with PHP without any hassles! Definitely the easiest set up I've tried.

I ran into a wall when using the LAMPApp plugin: http://probo.ci/docs/plugins/lamp-plugin/

I specified a databaseName and databaseUser, which seemed to have worked fine. But then I got stuck. There's no databasePassword! How am I supposed to access the created database if I don't have a password? I tried just omitting the password and using the username as the password, neither worked.

@quicksketch
Copy link
Author

quicksketch commented Apr 30, 2016

From reading through

addScriptCreateDatbase() {
, it seems I can short cut this entire username business by using the username root and the password strongpassword 🤦

I don't really mind the weak password (this is a temporary machine after all), but it's weird that there is a password at all. Why not just allow access to MySQL without a password entirely?

It doesn't look like the specified databaseUser is used at all, as the account is never created nor granted permissions (though the root user is).

@tizzo
Copy link
Contributor

tizzo commented Jun 23, 2016

That's a good point - we should probably create database users and passwords.

Is there a way to set no password? I know you can create a .my.conf file to prefill it with the cli client but I don't think there's a way to specify that no password is required. Though we could make the password an empty string which is roughly equivalent.

@jodyHamilton
Copy link
Contributor

#150

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants