Merge pull request #44 from freshworks/redisinsights-fix

fix in scan command and support for redis insights
freshworks · Dec 5, 2024 · 0c8e0b6 · 0c8e0b6
2 parents 4db4a88 + 92ac153
commit 0c8e0b6
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/source/extensions/filters/network/redis_proxy/command_splitter_impl.cc b/source/extensions/filters/network/redis_proxy/command_splitter_impl.cc
@@ -1383,7 +1383,7 @@ SplitRequestPtr ScanRequest::create(Router& router, Common::Redis::RespValuePtr&
 
   // Iterate over the arguments and validate
   for (size_t i = 2; i < incoming_request->asArray().size(); i++) {
-    std::string arg = incoming_request->asArray()[i].asString();
+    std::string arg = absl::AsciiStrToLower(incoming_request->asArray()[i].asString());
     std::string value = incoming_request->asArray()[++i].asString();
 
     if (!validateArgument(arg, value)) {