Fix review issues

Fixes review issues

- Unecessary None default in .get()
- Defining new fmt strings as optional

Author: barneysowood

salt/_logging/impl.py

@@ -248,23 +248,21 @@ def _log(
         if extra is None:
             extra = {}
 
-        current_jid = (
-            salt.utils.ctx.get_request_context().get("data", {}).get("jid", None)
-        )
+        current_jid = salt.utils.ctx.get_request_context().get("data", {}).get("jid")
         log_fmt_jid = (
             salt.utils.ctx.get_request_context()
             .get("opts", {})
             .get("log_fmt_jid", None)
         )
 
         current_minion_id = (
-            salt.utils.ctx.get_request_context().get("data", {}).get("id", None)
+            salt.utils.ctx.get_request_context().get("data", {}).get("id")
         )
 
         log_fmt_minion_id = (
             salt.utils.ctx.get_request_context()
             .get("opts", {})
-            .get("log_fmt_minion_id", None)
+            .get("log_fmt_minion_id")
         )
 
         if current_jid is not None:

salt/config/__init__.py

@@ -352,9 +352,9 @@ def _gather_buffer_space():
         # The format for a given log file
         "log_fmt_logfile": (tuple, str),
         # The format for JIDs prior to formatting into log lines as %(jid)s
-        "log_fmt_jid": str,
+        "log_fmt_jid": (type(None), str),
         # The format for minion_ids prior to formatting into log lines as %(minion_id)s
-        "log_fmt_minion_id": str,
+        "log_fmt_minion_id": (type(None), str),
         # A dictionary of logging levels
         "log_granular_levels": dict,
         # The maximum number of bytes a single log file may contain before