diff options
author | Shay Fadida <shayfadida@gmail.com> | 2022-09-21 08:10:03 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-09-21 08:10:03 +0300 |
commit | eedb8b172474dd7776d9bbb0f2954a1394027289 (patch) | |
tree | 7edb11c9a8ebdd3eb8e76839567586b7f012319d | |
parent | c2b0c13d5c0fab49131f6f5e844f80bfa43f6219 (diff) | |
download | redis-eedb8b172474dd7776d9bbb0f2954a1394027289.tar.gz |
Fix missing sections for INFO ALL with module (#11291)
When using `INFO ALL <section>`, when `section` is a specific module section.
Redis will not print the additional section(s).
The fix in this case, will search the modules info sections if the user provided additional sections to `ALL`.
Co-authored-by: Oran Agra <oran@redislabs.com>
-rw-r--r-- | src/server.c | 10 | ||||
-rw-r--r-- | tests/unit/moduleapi/infotest.tcl | 8 |
2 files changed, 15 insertions, 3 deletions
diff --git a/src/server.c b/src/server.c index 797c79fc3..a1f8eca78 100644 --- a/src/server.c +++ b/src/server.c @@ -6010,9 +6010,13 @@ sds genRedisInfoString(dict *section_dict, int all_sections, int everything) { } /* Get info from modules. - * if user asked for "everything" or "modules", or a specific section - * that's not found yet. */ - if (everything || dictFind(section_dict, "modules") != NULL || sections < (int)dictSize(section_dict)) { + * Returned when the user asked for "everything", "modules", or a specific module section. + * We're not aware of the module section names here, and we rather avoid the search when we can. + * so we proceed if there's a requested section name that's not found yet, or when the user asked + * for "all" with any additional section names. */ + if (everything || dictFind(section_dict, "modules") != NULL || sections < (int)dictSize(section_dict) || + (all_sections && dictSize(section_dict))) + { info = modulesCollectInfo(info, everything || dictFind(section_dict, "modules") != NULL ? NULL: section_dict, diff --git a/tests/unit/moduleapi/infotest.tcl b/tests/unit/moduleapi/infotest.tcl index 354487a19..ccd8c4ecb 100644 --- a/tests/unit/moduleapi/infotest.tcl +++ b/tests/unit/moduleapi/infotest.tcl @@ -44,6 +44,14 @@ start_server {tags {"modules"}} { assert { [string match "*used_memory*" $info] } } + test {module info all infotest} { + set info [r info all infotest] + # info all infotest should contain both ALL and the module information + assert { [string match "*Spanish*" $info] } + assert { [string match "*infotest_*" $info] } + assert { [string match "*used_memory*" $info] } + } + test {module info everything} { set info [r info everything] # info everything contains all default sections, but not ones for crash report |