🐛 Filename() panics when non-file URI is passed (konveyor#575) (konve…

…yor#576) Signed-off-by: Pranav Gaikwad <[email protected]> Signed-off-by: Cherry Picker <[email protected]> Signed-off-by: Pranav Gaikwad <[email protected]> Signed-off-by: Cherry Picker <[email protected]> Co-authored-by: Pranav Gaikwad <[email protected]>
pranavgaikwad · Apr 15, 2024 · 3e35a11 · 3e35a11
1 parent fb7c81e
commit 3e35a11
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/engine/engine.go b/engine/engine.go
@@ -4,6 +4,7 @@ import (
 	"bufio"
 	"context"
 	"fmt"
+	"net/url"
 	"os"
 	"path/filepath"
 	"regexp"
@@ -414,6 +415,10 @@ func processRule(ctx context.Context, rule Rule, ruleCtx ConditionContext, log l
 func (r *ruleEngine) getRelativePathForViolation(fileURI uri.URI) (uri.URI, error) {
 	var sourceLocation string
 	if fileURI != "" {
+		u, err := url.ParseRequestURI(string(fileURI))
+		if err != nil || u.Scheme != uri.FileScheme {
+			return fileURI, nil
+		}
 		file := fileURI.Filename()
 		// get the correct source from config
 		for _, locationPrefix := range r.locationPrefixes {