🐛 Filename() panics when non-file URI is passed

Signed-off-by: Pranav Gaikwad <pgaikwad@redhat.com>
konveyor · Apr 15, 2024 · 3d71f9a · 3d71f9a
1 parent ef5b9ec
commit 3d71f9a
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
 		for _, locationPrefix := range r.locationPrefixes {