Skip to content

Commit 9321f90

Browse files
refactor(alpine): remove type assertion when reading the APKINDEX archive (#11013)
Co-authored-by: lxcxjxhx <lxcxjxhx@users.noreply.github.com> Co-authored-by: DmitriyLewen <dmitriy.lewen@smartforce.io>
1 parent 6740bb8 commit 9321f90

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

  • pkg/fanal/analyzer/imgconf/apk

pkg/fanal/analyzer/imgconf/apk/apk.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -99,13 +99,13 @@ func (a alpineCmdAnalyzer) fetchApkIndexArchive(targetOS types.OS) (*apkIndex, e
9999

100100
url := fmt.Sprintf(a.apkIndexArchiveURL, osVer)
101101
var reader io.Reader
102-
if strings.HasPrefix(url, "file://") {
103-
var err error
104-
reader, err = builtinos.Open(strings.TrimPrefix(url, "file://"))
102+
if filePath, ok := strings.CutPrefix(url, "file://"); ok {
103+
f, err := builtinos.Open(filePath)
105104
if err != nil {
106105
return nil, xerrors.Errorf("failed to read APKINDEX archive file: %w", err)
107106
}
108-
defer reader.(*builtinos.File).Close()
107+
defer f.Close()
108+
reader = f
109109
} else {
110110
// nolint
111111
resp, err := http.Get(url)

0 commit comments

Comments
 (0)