Reture ParseError while auth failed.

src/brpc/policy/redis_protocol.cpp

@@ -85,17 +85,16 @@ ParseResult ParseRedisMessage(butil::IOBuf* source, Socket* socket,
         }
 
         if (pi.with_auth) {
-            if (msg->response.reply_size() != 1) {
-                socket->GivebackPipelinedInfo(pi);
-                return MakeParseError(PARSE_ERROR_ABSOLUTELY_WRONG);
+            if (msg->response.reply_size() != 1 ||
+                (!(msg->response.reply(0).type() == brpc::REDIS_REPLY_STATUS &&
+                  msg->response.reply(0).data().compare("OK") == 0))) {
+                LOG(ERROR) << "Redis Auth failed: " << msg->response;
+                return MakeParseError(PARSE_ERROR_NO_RESOURCE, 
+                                      "Fail to authenticate with Redis");
             }
 
             DestroyingPtr<InputResponse> auth_msg(
                  static_cast<InputResponse*>(socket->release_parsing_context()));
-            if (!(auth_msg->response.reply(0).type() == brpc::REDIS_REPLY_STATUS &&
-                  auth_msg->response.reply(0).data().compare("OK") == 0)) {
-                LOG(ERROR) << "Redis Auth failed: " << auth_msg->response.reply(0);
-            }
             pi.with_auth = false;
             continue;
         }