community.general
community.general copied to clipboard
feat(lookup/bitwarden): add support for fetching all items from a collection
SUMMARY
At the moment the lookup plugin allows to fetch items matching a mandatory search value. It will be useful to retrieve the full item list for a given collection.
This PR allows to fetch all items of a given collection id, by letting search_value optional when collection_id is provided.
ISSUE TYPE
- Feature Pull Request
COMPONENT NAME
community.general.bitwarden - lookup
ADDITIONAL INFORMATION
bwcli documentation: https://bitwarden.com/help/cli/#list
The test ansible-test sanity --test pep8 [explain] failed with 1 error:
plugins/lookup/bitwarden.py:134:38: E231: missing whitespace after ','
The test ansible-test sanity --test pep8 [explain] failed with 1 error:
plugins/lookup/bitwarden.py:134:38: E231: missing whitespace after ','
The test ansible-test sanity --test pep8 [explain] failed with 1 error:
plugins/lookup/bitwarden.py:134:38: E231: missing whitespace after ','
The test ansible-test sanity --test pep8 [explain] failed with 1 error:
plugins/lookup/bitwarden.py:134:38: E231: missing whitespace after ','
cc @lungj click here for bot help
Please note that this PR now has a conflict.
If nobody objects, I'll merge this in ~a week.
Backport to stable-8: 💚 backport PR created
✅ Backport PR branch: patchback/backports/stable-8/f5fa16c881b94f87e19b4d75b300db3b6f72cd20/pr-8013
Backported as https://github.com/ansible-collections/community.general/pull/8140
🤖 @patchback I'm built with octomachinery and my source is open — https://github.com/sanitizers/patchback-github-app.
@neilime thanks for your contribution!