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

reflex build with placeholder email doesn't update reflex.yaml #226

Open
mcbanderson opened this issue Sep 5, 2020 · 2 comments
Open

reflex build with placeholder email doesn't update reflex.yaml #226

mcbanderson opened this issue Sep 5, 2020 · 2 comments

Comments

@mcbanderson
Copy link
Contributor

Version: v1.2.2

When running reflex build with the placeholder contact email, you are prompted if you want to update the email address. If you select yes and then enter a new email address, the entered email address is used for the built terraform files but reflex.yaml is not updated.

@joerk2313
Copy link
Contributor

Would the file Function work?

Example:

file("${path.module}/hello.txt")
Hello World

https://www.terraform.io/docs/configuration/functions/file.html

@mcbanderson
Copy link
Contributor Author

No, the file function only reads the contents of a file. And since that's a Terraform function we wouldn't be able to utilize it from the CLI anyways.

The relevant code is here: https://github.com/reflexivesecurity/reflex-cli/blob/master/reflex_cli/commands/command_build.py#L54-L77

After the configuration is updated on line 77 we'd then need to write that configuration to the configuration file, similar to what we do with reflex update.

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