fix: grpc path should start with http(s)

Author: piket

go/internal/feast/registry/grpc.go

@@ -81,11 +81,11 @@ func buildTLSCredentials(certPath string) (credentials.TransportCredentials, err
 // parseGrpcTarget strips grpc:// or grpcs:// scheme prefixes and returns
 // the target address along with whether TLS should be used.
 func parseGrpcTarget(path string) (target string, useTLS bool) {
-	if strings.HasPrefix(path, "grpcs://") {
-		return strings.TrimPrefix(path, "grpcs://"), true
+	if strings.HasPrefix(path, "https://") {
+		return strings.TrimPrefix(path, "https://"), true
 	}
-	if strings.HasPrefix(path, "grpc://") {
-		return strings.TrimPrefix(path, "grpc://"), false
+	if strings.HasPrefix(path, "http://") {
+		return strings.TrimPrefix(path, "http://"), false
 	}
 	return path, false
 }

go/internal/feast/registry/grpc_test.go

@@ -175,12 +175,12 @@ func TestGrpcTeardown(t *testing.T) {
 
 func TestParseGrpcTarget(t *testing.T) {
 	cases := []struct {
-		input    string
-		target   string
-		useTLS   bool
+		input  string
+		target string
+		useTLS bool
 	}{
-		{"grpc://host:50051", "host:50051", false},
-		{"grpcs://host:50051", "host:50051", true},
+		{"http://host:50051", "host:50051", false},
+		{"https://host:50051", "host:50051", true},
 		{"host:50051", "host:50051", false},
 	}
 	for _, c := range cases {