Remove passwords when logging the conf details. Fix typos.

Author: erezsh

README.md

@@ -211,7 +211,7 @@ Reasons to use a configuration file:
 
 - Convenience - Set-up the parameters for diffs that need to run often
 
-- Easier and more readible - You can define the database connection settings as config values, instead of in a URI.
+- Easier and more readable - you can define the database connection settings as config values, instead of in a URI.
 
 - Gives you fine-grained control over the settings switches, without requiring any Python code.
 

data_diff/__main__.py

@@ -1,3 +1,4 @@
+from copy import deepcopy
 import sys
 import time
 import json
@@ -25,6 +26,13 @@
     "-": "red",
 }
 
+def _remove_passwords_in_dict(d: dict):
+    for k, v in d.items():
+        if k == 'password':
+            d[k] = '*' * len(v)
+        elif isinstance(v, dict):
+            _remove_passwords_in_dict(v)
+
 
 @click.command()
 @click.argument("database1", required=False)
@@ -110,6 +118,8 @@ def _main(
     if debug:
         logging.basicConfig(level=logging.DEBUG, format=LOG_FORMAT, datefmt=DATE_FORMAT)
         if __conf__:
+            __conf__ = deepcopy(__conf__)
+            _remove_passwords_in_dict(__conf__)
             logging.debug(f"Applied run configuration: {__conf__}")
     elif verbose:
         logging.basicConfig(level=logging.INFO, format=LOG_FORMAT, datefmt=DATE_FORMAT)