Conversation
|
+1 on this, I've been using this branch for several months because I like to store developer ssh keys in my user data bags. |
|
@fnichol What do you think? +1 |
|
👍 |
4 similar comments
|
+1 |
|
+1 |
|
+1 |
|
+1 |
|
@maksar thanks for the PR and very sorry for getting back so late. |
theckman
left a comment
There was a problem hiding this comment.
I think there is a lot of value in getting this merged-in to the LWRP. I agree with Ranjib's feedback and would like to see the data_bag_encrption_key attribute removed.
I know this has been outstanding for awhile, so if you aren't able to make these changes please let us know.
| name = i.gsub(/[.]/, '-') | ||
|
|
||
| u = if node['user']['data_bag_encrypted'] | ||
| Chef::EncryptedDataBagItem.load(bag, name, node['user']['data_bag_encryption_key']) |
There was a problem hiding this comment.
We should rely on the key being provided in the client's config and not here. We should remove any references to this attribute:
Chef::EncryptedDataBagItem.lad(bag, name)
By using
"data_bag_encrypted" : trueit is now possible to use encrypted data bags.