Commit 8b7224cf authored by Linus Nordberg's avatar Linus Nordberg

Don't treat exit 10 from dynamic scripts differently from any other non-zero code.

clientwr() should treat the dynamic lookup as a failure and will not
be any happier to know that the exact error was that it didn't resolv.
The script can do whatever logging is wanted.

That said, this commit also makes the scripts exit with 10 in order to
signal failure.
parent 39e90a8c
......@@ -2297,14 +2297,9 @@ int dynamicconfig(struct server *server) {
}
if (status) {
if (WEXITSTATUS(status) == 10) {
debug(DBG_INFO, "dynamicconfig: command signals empty config");
}
else {
debug(DBG_INFO, "dynamicconfig: command exited with status %d",
WEXITSTATUS(status));
goto errexit;
}
debug(DBG_INFO, "dynamicconfig: command exited with status %d",
WEXITSTATUS(status));
goto errexit;
}
if (ok)
......
......@@ -70,4 +70,4 @@ if [ -n "${SERVERS}" ]; then
exit 0
fi
exit 0
exit 10 # No server found.
......@@ -48,4 +48,4 @@ if test -n "${SERVERS}" ; then
exit 0
fi
exit 0
exit 10 # No server found.
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment