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

move IAM service configuration to a separate config file #348

Open
benmcclelland opened this issue Dec 19, 2023 · 2 comments
Open

move IAM service configuration to a separate config file #348

benmcclelland opened this issue Dec 19, 2023 · 2 comments
Labels
enhancement New feature or request IAM account related features/bugs

Comments

@benmcclelland
Copy link
Member

Describe the solution you'd like
The IAM service options are getting a bit overwhelming in the help output. Since we anticipate several more IAM service integrations in the future, lets move the IAM service configuration out of the cli args and into a separate config file.

The goal here is to reduce the cli options to just the following:

--iam-dir
--iam-config

We can leave the internal IAM service support with --iam-dir option for easier testing, but all others should be configured through a config file specified with --iam-config

@benmcclelland benmcclelland added enhancement New feature or request IAM account related features/bugs labels Dec 19, 2023
@benmcclelland
Copy link
Member Author

It looks like TOML might be a good format for intuitive config file management: https://godocs.io/github.com/BurntSushi/toml

@benmcclelland
Copy link
Member Author

We may also want to consider that some options could be set by environment variables as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request IAM account related features/bugs
Projects
None yet
Development

No branches or pull requests

1 participant