curtin-dev team mailing list archive

[Dan Bungert <mp+452998@xxxxxxxxxxxxxxxxxx>]

Dan Bungert has proposed merging ~dbungert/curtin:logs into curtin:master.

Commit message:
install: do not log merged config

Requested reviews:
  curtin developers (curtin-dev)

For more details, see:
https://code.launchpad.net/~dbungert/curtin/+git/curtin/+merge/452998
-- 
Your team curtin developers is requested to review the proposed merge of ~dbungert/curtin:logs into curtin:master.

diff --git a/curtin/commands/install.py b/curtin/commands/install.py
index 3309c63..9fb4116 100644
--- a/curtin/commands/install.py
+++ b/curtin/commands/install.py
@@ -453,7 +453,8 @@ def cmd_install(args):
 
     LOG.info(INSTALL_START_MSG)
     LOG.debug('LANG=%s', os.environ.get('LANG'))
-    LOG.debug("merged config: %s" % cfg)
+    # disabled - logs wifi PSK and probably other things to syslog
+    # LOG.debug("merged config: %s" % cfg)
 
     migrate_proxy_settings(cfg)
     for k in ('http_proxy', 'https_proxy', 'no_proxy'):