Quoted parameters should not be split on spaces

jhaals / ansible-vault

ansible lookup plugin for secrets stored in Vault(by HashiCorp)

BSD 3-Clause "New" or "Revised" License

347 stars 65 forks source link

Closed ajrgrubbs closed 7 years ago

ajrgrubbs commented 7 years ago

Example:

lookup('vault', 'my/arbitrary/path param="My short sentence"')

This would resolve param to be "My. (Then an index error occurs when resolving the next term, so nothing is submitted)

I think this should be an easy fix using the shlex module's split() method. I'd be happy to submit a PR but I'd like your input first.

jhaals commented 7 years ago

Sounds like a good idea to support quoted parameters 👍 👍