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

Sensitive data in Chef logs #370

Closed
isuftin opened this issue Nov 8, 2016 · 3 comments
Closed

Sensitive data in Chef logs #370

isuftin opened this issue Nov 8, 2016 · 3 comments

Comments

@isuftin
Copy link
Contributor

isuftin commented Nov 8, 2016

Sensitive data is placed into Chef logs.

Example from Kitchen test run with throwaway sensitive data:

         * consul_config[consul] action create
           * directory[/etc/consul] action create (up to date)
           * directory[/etc/consul/conf.d] action create (up to date)
           * file[/etc/consul/consul.json] action create
             - update content in file /etc/consul/consul.json from 26502e to 03906c
             --- /etc/consul/consul.json        2016-11-08 19:16:39.630048962 +0000
             +++ /etc/consul/.chef-consul20161108-16435-5eox0q.json     2016-11-08 19:16:48.815048760 +0000
             @@ -1,5 +1,6 @@
       {
         "acl_datacenter": "kitchen",
             +  "acl_master_token": "67f72b33-730b-4217-856e-08b7cd0d67cf",
         "addresses": {
           "https": "0.0.0.0"
         },
             @@ -13,6 +14,7 @@
         "datacenter": "kitchen",
         "disable_remote_exec": true,
         "enable_syslog": true,
             +  "encrypt": "Q7EuxolHM+NawIgFkXegIA==",
         "key_file": "/tmp/kitchen/data/rootCA.key",
         "node_name": "kitchen_server",
         "ports": {
             - restore selinux security context

Expected behavior is for the configuration writing to take the behavior of the Chef template resource using the "sensitive" flag. Do not write out to log what is being written to the configuration files.

@empath
Copy link

empath commented Nov 18, 2016

Yeah, this is a big deal if you're running it with jenkins.

@legal90
Copy link
Contributor

legal90 commented Dec 16, 2016

Fixed by #376

@legal90 legal90 closed this as completed Dec 16, 2016
@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants