fix: spdx element id cleanup

manifestori · Feb 25, 2024 · 5a51626 · 5a51626
1 parent ee68c08
commit 5a51626
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/parser/save.go b/parser/save.go
@@ -8,6 +8,7 @@ import (
 	"io"
 	"os"
 	"slices"
+	"strings"
 
 	spdx_json "github.com/spdx/tools-golang/json"
 	"github.com/spdx/tools-golang/spdx"
@@ -124,6 +125,15 @@ func getRootPackageIndex(doc *Document) (int, int) {
 	return i, j
 }
 
+func sanitizeDocumentName(name string) string {
+	var str = strings.ToLower(name)
+	str = strings.ReplaceAll(str, " ", "-")
+	str = strings.ReplaceAll(str, "/", "-")
+	str = strings.ReplaceAll(str, "\\", "-")
+	str = strings.ReplaceAll(str, "_", "-")
+	return str
+}
+
 func updateRelationships(doc *Document, composableDocs []*Document) (*Document, []*Document) {
 	for _, cdoc := range composableDocs {
 		i, j := getRootPackageIndex(cdoc)
@@ -134,7 +144,7 @@ func updateRelationships(doc *Document, composableDocs []*Document) (*Document,
 		if rootPkg == nil {
 			rootPkg = &spdx.Package{
 				PackageName:           cdoc.SPDXDocRef.DocumentName,
-				PackageSPDXIdentifier: spdx_common.MakeDocElementID("", cdoc.SPDXDocRef.DocumentName).ElementRefID,
+				PackageSPDXIdentifier: spdx_common.MakeDocElementID("", sanitizeDocumentName(cdoc.SPDXDocRef.DocumentName)).ElementRefID,
 			}
 			cdoc.SPDXDocRef.Packages = append([]*spdx.Package{rootPkg}, cdoc.SPDXDocRef.Packages...)