Skip to content

add support for reading key from remote url#77

Merged
seth merged 1 commit into
chef:masterfrom
lusis:topic/remote-databag-keys
May 4, 2011
Merged

add support for reading key from remote url#77
seth merged 1 commit into
chef:masterfrom
lusis:topic/remote-databag-keys

Conversation

@lusis

@lusis lusis commented May 2, 2011

Copy link
Copy Markdown
Contributor

This adds support for reading encryption keys from a remote url (similar to my existing cookbook). Not sure how you wanted to handle error so I made something up as argument errors. Also didn't see any functional tests for the encryption stuff so I just ran the existing spec. Here's a sample session output:

[] me@~/development/chef-repo$ knife data bag show --secret-file http://localhost:5678/ephemerals/my_company/decrypt test_edb passwords
ERROR: ArgumentError: Remote key not available from 'http://localhost:5678/ephemerals/my_company/decrypt'
[] me@~/development/chef-repo$ knife data bag show --secret-file http://localhost:5678/ephemerals/my_company test_edb passwords
ERROR: ArgumentError: Remote key not found at 'http://localhost:5678/ephemerals/my_company'
[] me@~/development/chef-repo$ knife data bag show --secret-file http://localhost:5678/ephemerals/my_company/decrypt test_edb passwords
id:     passwords
mysql:  MySexyPassword
[] me@~/development/chef-repo$

@seth seth merged commit efdd525 into chef:master May 4, 2011
@seth

seth commented May 4, 2011

Copy link
Copy Markdown
Contributor

I merged this in 52ab7df which also includes a bit of refactoring, some tests, and a fix on top.

Note: s[-1] == "\n" is not what you think.

lamont-granquist added a commit that referenced this pull request Sep 30, 2015
fixes for omnibus-ruby 1.1.0
srenatus pushed a commit to srenatus/chef that referenced this pull request Apr 5, 2016
fix the issue76 missing version from the cookbook delete path
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants