Ignore setting PerRPCCreds as clientopts (#527)

When PerRPCCreds is used, we are already setting it as part of
DialParams, so we don't have to once again include it as part of
clientopts when creating the connection.
When included again as part of clientopts, it results in gRPC adding
duplicate Authorization header to the requests.

Issue: bazelbuild/reclient#27 (thanks to
@MarshallOfSound for debugging this)

Tested: I build with this, ran a sample command and confirmed with
mitmproxy that authorization header wasn't repeated.

Author: gkousik

go/pkg/flags/flags.go

@@ -106,12 +106,16 @@ func NewClientFromFlags(ctx context.Context, opts ...client.Opt) (*client.Client
 		opts = append(opts, client.RPCTimeouts(timeouts))
 	}
 	var perRPCCreds *client.PerRPCCreds
+	tOpts := []client.Opt{}
 	for _, opt := range opts {
 		switch opt.(type) {
 		case *client.PerRPCCreds:
 			perRPCCreds = (opt).(*(client.PerRPCCreds))
+		default:
+			tOpts = append(tOpts, opt)
 		}
 	}
+	opts = tOpts
 
 	dialOpts := make([]grpc.DialOption, 0)
 	if *KeepAliveTime > 0*time.Second {