From d6b30e43cd7ebb053f047cc5d3ef75f4be309390 Mon Sep 17 00:00:00 2001
From: Jingyi Hu <jingyih@google.com>
Date: Tue, 12 Feb 2019 15:40:39 -0800
Subject: [PATCH] etcdserver: remove auth validation loop

Remove auth validation loop in v3_server.raftRequest(). Re-validation
when error ErrAuthOldRevision occurs should be handled on client side.
---
 etcdserver/v3_server.go | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/etcdserver/v3_server.go b/etcdserver/v3_server.go
index bfe08ea35..70b7177d3 100644
--- a/etcdserver/v3_server.go
+++ b/etcdserver/v3_server.go
@@ -577,12 +577,7 @@ func (s *EtcdServer) raftRequestOnce(ctx context.Context, r pb.InternalRaftReque
 }
 
 func (s *EtcdServer) raftRequest(ctx context.Context, r pb.InternalRaftRequest) (proto.Message, error) {
-	for {
-		resp, err := s.raftRequestOnce(ctx, r)
-		if err != auth.ErrAuthOldRevision {
-			return resp, err
-		}
-	}
+	return s.raftRequestOnce(ctx, r)
 }
 
 // doSerialize handles the auth logic, with permissions checked by "chk", for a serialized request "get". Returns a non-nil error on authentication failure.