Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add vault support to copy action plugin via dataloader implementation #14079

Conversation

cambell-prince
Copy link
Contributor

Added two methods to dataloader, along the lines discussed with @bcoca in #13849.

  • get_real_file will decrypt vault encrypted files and return either a path to a temporary file or simply the path to the unencrypted file if its not encrypted.
  • cleanup_real_file will remove a temporary file created with a previous call to get_real_file

… dataloader

- get_real_file will decrypt vault encrypted files and return a path to
  a temporary file.

- cleanup_real_file will remove a temporary file created previously with
  get_real_file
@bcoca bcoca added this to the 2.1.0 milestone Jan 22, 2016
danielcompton added a commit to clojars/infrastructure that referenced this pull request Feb 2, 2016
Waiting on ansible/ansible#14079 to be merged,
we can use this in the meantime.
@bcoca bcoca added the needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html label Mar 8, 2016
@bcoca bcoca mentioned this pull request Apr 14, 2016
@bcoca
Copy link
Member

bcoca commented Apr 19, 2016

closing via merged #15417

@bcoca bcoca closed this Apr 19, 2016
@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_pull_request labels Mar 4, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature This issue/PR relates to a feature request. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants