mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
fix vault-keyring.py 'No [vault] section' error
vault-keyring.py was using an older version of the ansible.constants.load_config_file() API. The newer version returns a tuple, which caused the config load to fail and a catch all exception to blame it on a missing section. Update to new API, and catch the ConfigParser error specifically. Fixes #15984
This commit is contained in:
parent
650bfdce8f
commit
675561e116
1 changed files with 5 additions and 4 deletions
9
contrib/vault/vault-keyring.py
Normal file → Executable file
9
contrib/vault/vault-keyring.py
Normal file → Executable file
|
@ -47,16 +47,17 @@
|
||||||
import sys
|
import sys
|
||||||
import getpass
|
import getpass
|
||||||
import keyring
|
import keyring
|
||||||
|
import ConfigParser
|
||||||
|
|
||||||
|
|
||||||
import ansible.constants as C
|
import ansible.constants as C
|
||||||
|
|
||||||
|
|
||||||
def main():
|
def main():
|
||||||
parser = C.load_config_file()
|
(parser,config_path) = C.load_config_file()
|
||||||
try:
|
try:
|
||||||
username = parser.get('vault', 'username')
|
username = parser.get('vault', 'username')
|
||||||
except:
|
except ConfigParser.NoSectionError:
|
||||||
sys.stderr.write('No [vault] section configured\n')
|
sys.stderr.write('No [vault] section configured in config file: %s\n' % config_path)
|
||||||
sys.exit(1)
|
sys.exit(1)
|
||||||
|
|
||||||
if len(sys.argv) == 2 and sys.argv[1] == 'set':
|
if len(sys.argv) == 2 and sys.argv[1] == 'set':
|
||||||
|
|
Loading…
Add table
Reference in a new issue