SERVER-38676 Handle 'true' in early processing of enableTestCommands

1 files changed, 11 insertions, 5 deletions

diff --git a/src/mongo/db/server_options_helpers.cpp b/src/mongo/db/server_options_helpers.cpp
index ae17cce3da2..145a719e015 100644
--- a/src/mongo/db/server_options_helpers.cpp
+++ b/src/mongo/db/server_options_helpers.cpp
@@ -288,12 +288,18 @@ Status validateBaseOptions(const moe::Environment& params) {
     }
 
     if (params.count("setParameter")) {
-        std::map<std::string, std::string> parameters =
-            params["setParameter"].as<std::map<std::string, std::string>>();
+        const auto parameters = params["setParameter"].as<std::map<std::string, std::string>>();
+
+        const bool enableTestCommandsValue = ([&parameters] {
+            const auto etc = parameters.find("enableTestCommands");
+            if (etc == parameters.end()) {
+                return false;
+            }
+            const auto& val = etc->second;
+            return val.compare("1") || val.compare("true");
+        })();
 
-        auto enableTestCommandsParameter = parameters.find("enableTestCommands");
-        if (enableTestCommandsParameter != parameters.end() &&
-            enableTestCommandsParameter->second.compare("1") == 0) {
+        if (enableTestCommandsValue) {
             // Only register failpoint server parameters if enableTestCommands=1.
             getGlobalFailPointRegistry()->registerAllFailPointsAsServerParameters();
         } else {