From abefd8e65247a30c03227bcefe9d95b1a3049b4c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Prpi=C4=8D?= Date: Fri, 30 Aug 2024 14:59:20 -0400 Subject: [PATCH] Support multiple purl identifiers in product_identification_helper This allows a vendor to specify multiple purl identifiers for a single component (present as a product version branch in the product tree). Multiple purls may identify the same component but point to different locations from where that component may be available. Thus, it is mandatory that if multiple purls are present in a single product_identification_helper object, they must only differ in their qualifiers. Otherwise they should be set up as different product tree branches. --- csaf_2.1/json_schema/csaf_json_schema.json | 20 +++++---- .../edit/etc/section-display-to-label.json | 2 +- .../edit/etc/section-label-to-display.json | 8 ++-- ...a-elements-01-defs-03-full-product-name.md | 42 ++++++++++--------- .../prose/edit/src/tests-01-mndtr-13-purl.md | 32 ++++++++++++-- 5 files changed, 69 insertions(+), 35 deletions(-) diff --git a/csaf_2.1/json_schema/csaf_json_schema.json b/csaf_2.1/json_schema/csaf_json_schema.json index 6cdd283c0..419660ff8 100644 --- a/csaf_2.1/json_schema/csaf_json_schema.json +++ b/csaf_2.1/json_schema/csaf_json_schema.json @@ -247,13 +247,19 @@ "minLength": 1 } }, - "purl": { - "title": "package URL representation", - "description": "The package URL (purl) attribute refers to a method for reliably identifying and locating software packages external to this specification.", - "type": "string", - "format": "uri", - "pattern": "^pkg:[A-Za-z\\.\\-\\+][A-Za-z0-9\\.\\-\\+]*\\/.+", - "minLength": 7 + "purls": { + "title": "List of package URLs", + "description": "Contains a list of package URLs (purl).", + "type": "array", + "minItems": 1, + "items": { + "title": "package URL representation", + "description": "The package URL (purl) attribute refers to a method for reliably identifying and locating software packages external to this specification.", + "type": "string", + "format": "uri", + "pattern": "^pkg:[A-Za-z\\.\\-\\+][A-Za-z0-9\\.\\-\\+]*/.+", + "minLength": 7 + } }, "sbom_urls": { "title": "List of SBOM URLs", diff --git a/csaf_2.1/prose/edit/etc/section-display-to-label.json b/csaf_2.1/prose/edit/etc/section-display-to-label.json index 354bf3960..bf89c8a4d 100644 --- a/csaf_2.1/prose/edit/etc/section-display-to-label.json +++ b/csaf_2.1/prose/edit/etc/section-display-to-label.json @@ -29,7 +29,7 @@ "3.1.3.3.1": "full-product-name-type-product-identification-helper-cpe", "3.1.3.3.2": "full-product-name-type-product-identification-helper-hashes", "3.1.3.3.3": "full-product-name-type-product-identification-helper-model-numbers", - "3.1.3.3.4": "full-product-name-type-product-identification-helper-purl", + "3.1.3.3.4": "full-product-name-type-product-identification-helper-purls", "3.1.3.3.5": "full-product-name-type-product-identification-helper-sbom-urls", "3.1.3.3.6": "full-product-name-type-product-identification-helper-serial-numbers", "3.1.3.3.7": "full-product-name-type-product-identification-helper-skus", diff --git a/csaf_2.1/prose/edit/etc/section-label-to-display.json b/csaf_2.1/prose/edit/etc/section-label-to-display.json index fa2c33010..5038c3bb5 100644 --- a/csaf_2.1/prose/edit/etc/section-label-to-display.json +++ b/csaf_2.1/prose/edit/etc/section-label-to-display.json @@ -94,7 +94,7 @@ "full-product-name-type-product-identification-helper-generic-uris": "3.1.3.3.8", "full-product-name-type-product-identification-helper-hashes": "3.1.3.3.2", "full-product-name-type-product-identification-helper-model-numbers": "3.1.3.3.3", - "full-product-name-type-product-identification-helper-purl": "3.1.3.3.4", + "full-product-name-type-product-identification-helper-purls": "3.1.3.3.4", "full-product-name-type-product-identification-helper-sbom-urls": "3.1.3.3.5", "full-product-name-type-product-identification-helper-serial-numbers": "3.1.3.3.6", "full-product-name-type-product-identification-helper-skus": "3.1.3.3.7", @@ -240,9 +240,9 @@ "vulnerabilities-property-ids": "3.2.4.6", "vulnerabilities-property-involvements": "3.2.4.7", "vulnerabilities-property-metrics": "3.2.4.8", - "vulnerabilities-property-metrics-content": "3.2.4.8.1", - "vulnerabilities-property-metrics-products": "3.2.4.8.2", - "vulnerabilities-property-metrics-source": "3.2.4.8.3", + "vulnerabilities-property-metrics-content": "3.2.4.8.1", + "vulnerabilities-property-metrics-products": "3.2.4.8.2", + "vulnerabilities-property-metrics-source": "3.2.4.8.3", "vulnerabilities-property-notes": "3.2.4.9", "vulnerabilities-property-product-status": "3.2.4.10", "vulnerabilities-property-references": "3.2.4.11", diff --git a/csaf_2.1/prose/edit/src/schema-elements-01-defs-03-full-product-name.md b/csaf_2.1/prose/edit/src/schema-elements-01-defs-03-full-product-name.md index 78bd55f56..978488d77 100644 --- a/csaf_2.1/prose/edit/src/schema-elements-01-defs-03-full-product-name.md +++ b/csaf_2.1/prose/edit/src/schema-elements-01-defs-03-full-product-name.md @@ -41,7 +41,7 @@ Product ID (`product_id`) holds a value of type Product ID (`product_id_t`). Helper to identify the product (`product_identification_helper`) of value type `object` provides in its properties at least one method which aids in identifying the product in an asset database. -Of the given eight properties `cpe`, `hashes`, `model_numbers`, `purl`, `sbom_urls`, `serial_numbers`, `skus`, +Of the given eight properties `cpe`, `hashes`, `model_numbers`, `purls`, `sbom_urls`, `serial_numbers`, `skus`, and `x_generic_uris`, one is mandatory. ``` @@ -51,27 +51,27 @@ and `x_generic_uris`, one is mandatory. "cpe": { // ... }, - "hashes": { + "hashes": [ // ... - }, - "model_numbers": { + ], + "model_numbers": [ // ... - }, - "purl": { + ], + "purls": [ // ... - }, - "sbom_urls": { + ], + "sbom_urls": [ // ... - }, - "serial_numbers": { + ], + "serial_numbers": [ // ... - }, - "skus": { + ], + "skus": [ // ... - }, - "x_generic_uris": { + ], + "x_generic_uris": [ // ... - } + ] } ``` @@ -238,9 +238,10 @@ Two `*` MUST NOT follow each other. IC25T060ATCS05-0 ``` -##### Full Product Name Type - Product Identification Helper - purl +##### Full Product Name Type - Product Identification Helper - purls -The package URL (purl) representation (`purl`) is a `string` of 7 or more characters with `pattern` (regular expression): +List of purls (`purls`) of value type `array` with 1 or more items contains a list of package URL (purl) identifiers. +A purl is a `string` of 7 or more characters that meets the criteria of `pattern` (regular expression): ``` ^pkg:[A-Za-z\\.\\-\\+][A-Za-z0-9\\.\\-\\+]*\\/.+ @@ -251,8 +252,11 @@ The package URL (purl) representation (`purl`) is a `string` of 7 or more charac > CSAF uses only the canonical form of purl to conform with section 3.3 of [cite](#RFC3986). > Therefore, URLs starting with `pkg://` are considered invalid. -This package URL (purl) attribute refers to a method for reliably identifying and locating software packages external to this specification. -See [cite](#PURL) for details. +A package URL (purl) attribute refers to a method for reliably identifying and locating software packages external to this specification. +See [cite](#PURL) for details. Multiple purls can be specified to allow for identifiers to locate identical components in different locations. + +If multiple purls are specified, they MUST only differ in their qualifiers. Otherwise, separate product branches should +be used to differentiate between the components. ##### Full Product Name Type - Product Identification Helper - SBOM URLs diff --git a/csaf_2.1/prose/edit/src/tests-01-mndtr-13-purl.md b/csaf_2.1/prose/edit/src/tests-01-mndtr-13-purl.md index 79262fca1..814bcfdf7 100644 --- a/csaf_2.1/prose/edit/src/tests-01-mndtr-13-purl.md +++ b/csaf_2.1/prose/edit/src/tests-01-mndtr-13-purl.md @@ -5,9 +5,9 @@ It MUST be tested that given purl is valid. The relevant paths for this test are: ``` - /product_tree/branches[](/branches[])*/product/product_identification_helper/purl - /product_tree/full_product_names[]/product_identification_helper/purl - /product_tree/relationships[]/full_product_name/product_identification_helper/purl + /product_tree/branches[](/branches[])*/product/product_identification_helper/purls[] + /product_tree/full_product_names[]/product_identification_helper/purls[] + /product_tree/relationships[]/full_product_name/product_identification_helper/purls[] ``` *Example 1 (which fails the test):* @@ -19,7 +19,9 @@ The relevant paths for this test are: "name": "Product A", "product_id": "CSAFPID-9080700", "product_identification_helper": { - "purl": "pkg:maven/@1.3.4" + "purls": [ + "pkg:maven/@1.3.4" + ] } } ] @@ -27,3 +29,25 @@ The relevant paths for this test are: ``` > Any valid purl has a name component. + +It MUST also be tested that if more than one purl is specified, they can only differ in their qualifiers. + +*Example 2 (which fails the test):* + +``` + "product_tree": { + "full_product_names": [ + { + "name": "Product A", + "product_id": "CSAFPID-9080700", + "product_identification_helper": { + "purls": [ + "pkg:maven/org.example.blue/logging@1.3.4", + "pkg:maven/org.example.blue/audit@1.3.4" + ] + } + } + ] + } +``` +> The two purls differ in the name component.