Skip to content

Commit

Permalink
monitor: fixed default value of 'user' config option
Browse files Browse the repository at this point in the history
1) man page explicitly and unconditionally says that default value
for this option is 'root' so this patch just aligns code with a doc

2) since at the moment "sssd running as non-root" feature isn't really
tested and is proposed at "use at your own risk" basis it wouldn't hurt
to require user to configure this option explicitly even when sssd is
built with "--with-sssd-user=sssd"

This should be changed when feature is really supported.

:relnote: default value of 'user' config option was fixed into accordance
with man page, i.e. default is 'root'

Reviewed-by: Sumit Bose <[email protected]>
  • Loading branch information
alexey-tikhonov authored and pbrezina committed Feb 19, 2021
1 parent 9aaa0e5 commit ee9dbea
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/monitor/monitor.c
Original file line number Diff line number Diff line change
Expand Up @@ -905,7 +905,7 @@ static int get_service_user(struct mt_ctx *ctx)

ret = confdb_get_string(ctx->cdb, ctx, CONFDB_MONITOR_CONF_ENTRY,
CONFDB_MONITOR_USER_RUNAS,
SSSD_USER, &user_str);
"root", &user_str);
if (ret != EOK) {
DEBUG(SSSDBG_FATAL_FAILURE, "Failed to get the user to run as\n");
return ret;
Expand Down

0 comments on commit ee9dbea

Please sign in to comment.