tests: Prevent syslog activity by slapd, take 2
authorAndres Freund
Fri, 17 Mar 2023 06:03:31 +0000 (23:03 -0700)
committerAndres Freund
Fri, 17 Mar 2023 06:18:31 +0000 (23:18 -0700)
Unfortunately it turns out that the logfile-only option added in b9f8d1cbad7
is only available in openldap starting in 2.6.

Luckily the option to control the log level (loglevel/-s) have been around for
much longer. As it turns out loglevel/-s only control what goes into syslog,
not what ends up in the file specified with 'logfile' and stderr.

While we currently are specifying 'logfile', nothing ends up in it, as the
option only controls debug messages, and we didn't set a debug level. The
debug level can only be configured on the commandline and also prevents
forking. That'd require larger changes, so this commit doesn't tackle that
issue.

Specify the syslog level when starting slapd using -s, as that allows to
prevent all syslog messages if one uses '0' instead of 'none', while loglevel
doesn't prevent the first message.

Discussion: https://postgr.es/m/20230311233708[email protected]
Backpatch: 11-

src/test/ldap/t/001_auth.pl

index 5131ba2345a11f9c88aefb22e18fc501158ffcb9..2019674ed4d0f02a754a37b00b0cd8176c182df2 100644 (file)
@@ -75,7 +75,6 @@ include $ldap_schema_dir/inetorgperson.schema
 
 pidfile $slapd_pidfile
 logfile $slapd_logfile
-logfile-only on
 
 access to *
         by * read
@@ -111,7 +110,8 @@ system_or_bail "openssl", "x509", "-req", "-in", "$slapd_certs/server.csr",
   "-CA", "$slapd_certs/ca.crt", "-CAkey", "$slapd_certs/ca.key",
   "-CAcreateserial", "-out", "$slapd_certs/server.crt";
 
-system_or_bail $slapd, '-f', $slapd_conf, '-h', "$ldap_url $ldaps_url";
+# -s0 prevents log messages ending up in syslog
+system_or_bail $slapd, '-f', $slapd_conf,'-s0', '-h', "$ldap_url $ldaps_url";
 
 END
 {