From 408d4fb35d3740ccffa06ec78d704dcc97eaee1c Mon Sep 17 00:00:00 2001 From: antirez Date: Thu, 23 Apr 2020 11:56:36 +0200 Subject: [PATCH] ACL: re-enable command execution of disabled users. After all I changed idea again: enabled/disabled should have a more clear meaning, and it only means: you can't authenticate with such user with new connections, however old connections continue to work as expected. --- src/acl.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/acl.c b/src/acl.c index bf5dd18f1..3194feb5b 100644 --- a/src/acl.c +++ b/src/acl.c @@ -1055,10 +1055,6 @@ int ACLCheckCommandPerm(client *c, int *keyidxptr) { /* If there is no associated user, the connection can run anything. */ if (u == NULL) return ACL_OK; - /* If the user is disabled we don't allow the execution of any - * command. */ - if (!(u->flags & USER_FLAG_ENABLED)) return ACL_DENIED_CMD; - /* Check if the user can execute this command. */ if (!(u->flags & USER_FLAG_ALLCOMMANDS) && c->cmd->proc != authCommand)