Following the fix for bug 14897, an smb.conf line like

     log level = 1 dsdb_group_json_audit:5@somewhere.log

no longer works properly; the '5' for dsdb_group_json_audit is effective, turning on the logging, but the '@somewhere.log' does nothing and the JSON logs end up in the main log file.

I am able to get 4.15.6 working by running setup_logging() again after the lpcfg_load():


--- a/lib/cmdline/cmdline_s4.c
+++ b/lib/cmdline/cmdline_s4.c
@@ -58,20 +58,33 @@ static bool _samba_cmdline_load_config_s4(void)
        default:
                break;
        }
 
        config_file = get_dyn_CONFIGFILE();
        ok = lpcfg_load(lp_ctx, config_file);
        if (!ok) {
                fprintf(stderr,
                        "Can't load %s - run testparm to debug it\n",
                        config_file);
 
                if (_require_smbconf) {
                        return false;
                }
        }
+       switch (_config_type) {
+       case SAMBA_CMDLINE_CONFIG_SERVER: {
+               /* 
+                * We need to setup_logging *again* to ensure multi-file
+                * logging is set up as specified in smb.conf.
+                */
+               setup_logging(getprogname(), DEBUG_FILE);
+               break;
+       }
+       default:
+               break;
+       }
 
        return true;
 }


but I am not sure whether it also still needs to be there *before* the lpcfg_load(). Ralph, Andreas, was there a specific reason for that order?