Skip to content

Commit 673b847

Browse files
Mixficsolwuxianrong
andauthored
fix: added the correct loading of admin-cmd-list in the configuration file (OpenAtomFoundation#3076)
* Added the correct loading of admin-cmd-list in the configuration file * add config to admin_cmd_list --------- Co-authored-by: wuxianrong <[email protected]>
1 parent 3c398f6 commit 673b847

2 files changed

Lines changed: 2 additions & 5 deletions

File tree

conf/pika.conf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ slow-cmd-list :
4747
# List of commands considered as administrative. These commands will be handled by the admin thread pool. Modify this list as needed.
4848
# Default commands: info, ping, monitor
4949
# This parameter is only supported by the CONFIG GET command and not by CONFIG SET.
50-
admin-cmd-list : info, ping, monitor
50+
admin-cmd-list : info, ping, monitor, auth, config
5151

5252
# The number of threads to write DB in slaveNode when replicating.
5353
# It's preferable to set slave's sync-thread-num value close to master's thread-pool-size.

src/pika_conf.cc

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -188,10 +188,7 @@ int PikaConf::Load() {
188188

189189
std::string admin_cmd_list;
190190
GetConfStr("admin-cmd-list", &admin_cmd_list);
191-
if (admin_cmd_list == ""){
192-
admin_cmd_list = "info, monitor, ping";
193-
SetAdminCmd(admin_cmd_list);
194-
}
191+
SetAdminCmd(admin_cmd_list);
195192

196193
std::string unfinished_full_sync;
197194
GetConfStr("internal-used-unfinished-full-sync", &unfinished_full_sync);

0 commit comments

Comments
 (0)