search

pierrecdn / phpipam

phpIPAM Docker image
GNU Lesser General Public License v3.0
94 stars 111 forks source link

Allow using Environment variable for DB Name #30

Closed timwsuqld closed 6 years ago

timwsuqld commented 6 years ago

Currently the MYSQL environment variables cover almost all the options, except for customising the DB name. 

$db['name'] = 'phpipam';

Can you please make it so we can customise that with environment variables as well.

Thanks

pierrecdn commented 6 years ago

Hi, Yes true it was done like that because it was convenient to use with the db setup done via the auto-installer, using the defaults. I can understand that for migration purposes and other use cases it could be nice... Any PR to submit? ;)

timwsuqld commented 6 years ago

Merge request submitted. I can't see any reason it would cause issues for the auto-installer how I've done it, as it'll still default to phpipam when the environment variable isn't set.