fix: don't clear qualifiers when ingesting CSAF

trustification · Oct 4, 2024 · 506e81b · 506e81b
1 parent daef8de
commit 506e81b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 8 deletions.
diff --git a/modules/fundamental/tests/advisory/csaf/reingest.rs b/modules/fundamental/tests/advisory/csaf/reingest.rs
@@ -113,13 +113,10 @@ async fn change_ps_list_vulns(ctx: &TrustifyContext) -> anyhow::Result<()> {
     );
     assert_eq!(purl.version.version, "1.76.0-4.redhat_00001.1.el9eap");
 
-    // FIXME(#879): I'd expect the qualifiers to be present, but they aren't.
-    /*
     assert_eq!(
         purl.qualifiers.clone().into_iter().collect::<Vec<_>>(),
         vec![("arch".to_string(), "noarch".to_string())]
     );
-    */
 
     // get vuln by purl
 
@@ -204,13 +201,10 @@ async fn change_ps_list_vulns_all(ctx: &TrustifyContext) -> anyhow::Result<()> {
     );
     assert_eq!(purl.version.version, "1.76.0-4.redhat_00001.1.el9eap");
 
-    // FIXME(#879): I'd expect the qualifiers to be present, but they aren't.
-    /*
     assert_eq!(
         purl.qualifiers.clone().into_iter().collect::<Vec<_>>(),
         vec![("arch".to_string(), "noarch".to_string())]
     );
-    */
 
     // get vuln by purl
 

diff --git a/modules/ingestor/src/service/advisory/csaf/creator.rs b/modules/ingestor/src/service/advisory/csaf/creator.rs
@@ -162,8 +162,7 @@ impl<'a> StatusCreator<'a> {
             }
 
             for purl in &product.packages {
-                let mut purl = purl.clone();
-                purl.qualifiers.clear();
+                let purl = purl.clone();
 
                 // Ingest purl status
                 let info = match purl.version.clone() {