Optimize FieldAccessKey

Author: WVerlaek

components/common-go/grpc/ratelimit.go

@@ -86,16 +86,20 @@ func NewRatelimitingInterceptor(f map[string]RateLimit) RatelimitingInterceptor
 }
 
 func fieldAccessKey(key string) keyFunc {
+	fields := strings.Split(key, ",")
+	paths := make([][]string, len(fields))
+	for i, field := range fields {
+		paths[i] = strings.Split(field, ".")
+	}
 	return func(req interface{}) (string, error) {
 		msg, ok := req.(proto.Message)
 		if !ok {
 			return "", status.Errorf(codes.Internal, "request was not a protobuf message")
 		}
 
-		fields := strings.Split(key, ",")
 		var composite string
-		for _, field := range fields {
-			val, ok := getFieldValue(msg.ProtoReflect(), strings.Split(field, "."))
+		for i, field := range fields {
+			val, ok := getFieldValue(msg.ProtoReflect(), paths[i])
 			if !ok {
 				return "", status.Errorf(codes.Internal, "Field %s does not exist in message. This is a rate limiting configuration error.", field)
 			}