search

tellerops / teller

Cloud native secrets management for developers - never leave your command line for secrets.
https://github.com/tellerops/teller
Apache License 2.0
2.8k stars 183 forks source link

bugfix(273): Teller file name inconsistencies #274

Closed stvnksslr closed 3 months ago

stvnksslr commented 3 months ago

reuse the variable declared for the default to prevent drift between teller new and file lookup

fixes #273

Rationale for changes to tests if the -c flag is passed teller should use this name/path and test to make sure its finding the correct one.

if no config is passed the config naming should follow the convention of teller new

❯ ls -lah | grep teller.yml
.rw-r--r--   182 stvnksslr 14 May 12:19  .teller.yml
.rw-r--r--   182 stvnksslr 14 May 12:51  teller.yml

teller on  bugfix/273 [!?] via 🦀 v1.79.0-nightly
❯ ./target/debug/teller -c teller.yml show
[aws_secretsmanager_1 (aws_secretsmanager)]: TOKEN = ***

teller on  bugfix/273 [!?] via 🦀 v1.79.0-nightly
❯ ./target/debug/teller show
[aws_secretsmanager_1 (aws_secretsmanager)]: TOKEN = ***
jondot commented 3 months ago

Looks great, thanks!