fix(build): add proper error message if path matcher has special chars

Signed-off-by: Yaroslav Pershin <62902094+iapershin@users.noreply.github.com>

Author: iapershin

pkg/path_matcher/common.go

@@ -1,13 +1,18 @@
 package path_matcher
 
 import (
-	"fmt"
+	"os"
 	"path/filepath"
 	"strings"
 
 	"github.com/bmatcuk/doublestar/v4"
 
 	"github.com/werf/common-go/pkg/util"
+	"github.com/werf/logboek"
+)
+
+const (
+	specialCharError = "The specified glob pattern contains special characters '?[{' that need to be properly used or escaped with '\\\\'."
 )
 
 func matchGlobs(pathPart string, globs []string) (inProgressGlobs, matchedGlobs []string) {
@@ -74,11 +79,14 @@ func isPathMatched(filePath, glob string) bool {
 		util.SafeTrimGlobsAndSlashesFromFilepath(glob),
 		filepath.Join(util.SafeTrimGlobsAndSlashesFromFilepath(glob), "**", "*"),
 	} {
-		matched, err := doublestar.PathMatch(escapeGlobSpecials(g), filePath)
+		matched, err := doublestar.PathMatch(g, filePath)
 		if err != nil {
-			panic(fmt.Sprintf("failed to match path %q with glob %q: %v", filePath, g, err))
+			logboek.Error().LogF("Failed to match path %q with glob %q: %v\n", filePath, g, err)
+			if strings.ContainsAny(g, "?[{") {
+				logboek.Error().LogF("%s\n", specialCharError)
+			}
+			os.Exit(1)
 		}
-
 		if matched {
 			return true
 		}
@@ -87,18 +95,6 @@ func isPathMatched(filePath, glob string) bool {
 	return false
 }
 
-func escapeGlobSpecials(path string) string {
-	specials := `?[]{}!\\`
-	var b strings.Builder
-	for _, r := range path {
-		if strings.ContainsRune(specials, r) {
-			b.WriteRune('\\')
-		}
-		b.WriteRune(r)
-	}
-	return b.String()
-}
-
 func formatPaths(paths []string) []string {
 	var result []string
 	for _, path := range paths {

pkg/path_matcher/common_test.go

@@ -14,7 +14,7 @@ func TestIsPathMatched(t *testing.T) {
 		{
 			name:     "exact match with special character",
 			filePath: "usr/bin/[",
-			glob:     "usr/bin/[",
+			glob:     "usr/bin/\\[",
 			want:     true,
 		},
 		{
@@ -47,6 +47,12 @@ func TestIsPathMatched(t *testing.T) {
 			glob:     "usr/bin",
 			want:     true,
 		},
+		{
+			name:     "glob with multiple patterns",
+			filePath: "images/img/Dockerfile",
+			glob:     "images/*/{Dockerfile,werf.inc.yaml}",
+			want:     true,
+		},
 	}
 
 	for _, tt := range tests {