Skip to content

Conversation

@crolfe
Copy link

@crolfe crolfe commented Feb 27, 2018

This fixes a bug that only occurs when ~/.vault-token is not present (i.e. Vault does not have a valid token).

From what I can tell, the data parameter this method takes is always a dict, so I .encode('utf-8') every value in the dict that this method is passed. This change naively assumes data will never be nested, which should be fine for now, but may need to be changed in the future.

The error message I was getting on master is below:
fatal: [10.10.19.108]: FAILED! => {"changed": false, "failed": true, "msg": "AnsibleError: An unhandled exception occurred while running the lookup plugin 'vault'. Error was a <class 'ansible.errors.AnsibleError'>, original message: Unable to retrieve personal token from vault: 'dict' object has no attribute 'encode'"}

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

Successfully merging this pull request may close these issues.

1 participant