fix

apache · May 27, 2024 · ef10245 · ef10245
1 parent 4c30a8b
commit ef10245
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/commands/cmd_server.cc b/src/commands/cmd_server.cc
@@ -29,6 +29,7 @@
 #include "config/config.h"
 #include "error_constants.h"
 #include "server/redis_connection.h"
+#include "server/redis_reply.h"
 #include "server/server.h"
 #include "stats/disk_stats.h"
 #include "storage/rdb.h"
@@ -740,7 +741,8 @@ class CommandHello final : public Commander {
       // kvrocks only supports REPL2 by now, but for supporting some
       // `hello 3`, it will not report error when using 3.
       if (protocol < 2 || protocol > 3) {
-        return {Status::NotOK, "NOPROTO unsupported protocol version"};
+        conn->Reply(redis::Error("NOPROTO unsupported protocol version"));
+        return Status::OK();
       }
     }