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

Environment variables included in the config file are not evaluated #1479

Closed
kenchan0130 opened this issue Dec 2, 2016 · 3 comments
Closed

Environment variables included in the config file are not evaluated #1479

kenchan0130 opened this issue Dec 2, 2016 · 3 comments

Comments

@kenchan0130
Copy link

@kenchan0130 kenchan0130 commented Dec 2, 2016

What version of Flyway are you using?

4.0.3

Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)

Command-line

What database are you using (type & version)?

MySQL 5.6.24

What operating system are you using?

Mac OSX 10.11.6

What did you do?

(Please include the content causing the issue, any relevant configuration settings, and the command you ran)

# test.conf
flyway.url=jdbc:mysql://${DATABASE_HOST}:3306/sample
flyway.user=root
flyway.locations=filesystem:db/migration
$ flyway -configFile=test.conf migrate
Flyway 4.0.3 by Boxfuse

Database password:
ERROR: Unable to obtain Jdbc connection from DataSource (jdbc:mysql://${DATABASE_HOST}:3306/sample) for user 'root': url parameter error '${DATABASE_HOST}' is a unknown parameter in the url jdbc:mysql://${DATABASE_HOST}:3306/sample
What did you expect to see?

I expected evaluating a DATABASE_HOST environment variable.
However, it occurred that error.

@axelfontaine axelfontaine added this to the Flyway 5.0 milestone Feb 1, 2017
@jadekler
Copy link

@jadekler jadekler commented Aug 18, 2017

Same problem here

@axelfontaine axelfontaine removed this from the Flyway 5.0.0 milestone Nov 27, 2017
@axelfontaine axelfontaine added this to the Flyway 5.1.0 milestone Nov 27, 2017
@axelfontaine axelfontaine removed this from the Flyway 5.1.0 milestone May 23, 2018
@axelfontaine axelfontaine added this to the Flyway 5.2.0 milestone May 23, 2018
@safouanenac
Copy link

@safouanenac safouanenac commented Jul 10, 2018

opened since 2 Dec 2016 !

@einnjo
Copy link

@einnjo einnjo commented Sep 18, 2018

I know flyway can be configured via placeholder environment variables but this means we would need to duplicate our database credential variables (once for flyway, once for app). Is there any way to get #1785 going again?

@axelfontaine axelfontaine removed this from the Flyway 5.2.0 milestone Sep 20, 2018
@axelfontaine axelfontaine added this to the Flyway 6.0.0 milestone Sep 20, 2018
@juliahayward juliahayward removed this from the Flyway 6.0.0 milestone Jul 31, 2019
@juliahayward juliahayward added this to the Flyway 6.1.0 milestone Jul 31, 2019
dohrayme pushed a commit to dohrayme/flyway that referenced this issue Feb 3, 2020
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

7 participants