Skip to content

Commit a197a84

Browse files
committed
Add PyPI OSV importer
Reference: #607 Signed-off-by: Ziad <ziadhany2016@gmail.com> add the necessary changes Signed-off-by: Ziad <ziadhany2016@gmail.com> remove aliases de-duplicate Signed-off-by: Ziad <ziadhany2016@gmail.com> reslove conflicts Signed-off-by: Ziad <ziadhany2016@gmail.com> add dateparser to setup.cfg Signed-off-by: Ziad <ziadhany2016@gmail.com> Resolving conflicts Signed-off-by: Ziad <ziadhany2016@gmail.com> resolve conflicts Signed-off-by: Ziad <ziadhany2016@gmail.com> add a require changes Signed-off-by: Ziad <ziadhany2016@gmail.com> add a require changes Signed-off-by: Ziad <ziadhany2016@gmail.com> add a require changes Signed-off-by: Ziad <ziadhany2016@gmail.com> add a require changes Signed-off-by: Ziad <ziadhany2016@gmail.com>
1 parent 47f6ae6 commit a197a84

7 files changed

Lines changed: 1050 additions & 0 deletions

File tree

requirements.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,3 +113,4 @@ wcwidth==0.2.5
113113
websocket-client==0.59.0
114114
yarl==1.7.2
115115
zipp==3.8.0
116+
dateparser==1.1.1

setup.cfg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ install_requires =
7575
lxml>=4.6.4
7676
defusedxml>=0.7.1
7777
Markdown>=3.3.0
78+
dateparser>=1.1.1
7879

7980
# networking
8081
GitPython>=3.1.17

vulnerabilities/helpers.py

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -267,3 +267,16 @@ def _get_gh_response(gh_token, graphql_query):
267267
endpoint = "https://api.github.com/graphql"
268268
headers = {"Authorization": f"bearer {gh_token}"}
269269
return requests.post(endpoint, headers=headers, json=graphql_query).json()
270+
271+
272+
def dedupe(original: List) -> List:
273+
"""
274+
Remove all duplicate items and return a new list preserving ordering
275+
>>> dedupe(["z","i","a","a","d","d"])
276+
['z', 'i', 'a', 'd']
277+
"""
278+
new_list = []
279+
for i in original:
280+
if i not in new_list:
281+
new_list.append(i)
282+
return new_list

vulnerabilities/importers/__init__.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,15 @@
2424
from vulnerabilities.importers import nginx
2525
from vulnerabilities.importers import nvd
2626
from vulnerabilities.importers import openssl
27+
from vulnerabilities.importers import pysec
2728

2829
IMPORTERS_REGISTRY = [
2930
nginx.NginxImporter,
3031
alpine_linux.AlpineImporter,
3132
github.GitHubAPIImporter,
3233
nvd.NVDImporter,
3334
openssl.OpensslImporter,
35+
pysec.PyPIImporter,
3436
]
3537

3638
IMPORTERS_REGISTRY = {x.qualified_name: x for x in IMPORTERS_REGISTRY}

vulnerabilities/importers/pysec.py

Lines changed: 240 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,240 @@
1+
# Copyright (c) 2017 nexB Inc. and others. All rights reserved.
2+
# http://nexb.com and https://github.com/nexB/vulnerablecode/
3+
# The VulnerableCode software is licensed under the Apache License version 2.0.
4+
# Data generated with VulnerableCode require an acknowledgment.
5+
#
6+
# You may not use this software except in compliance with the License.
7+
# You may obtain a copy of the License at: http://apache.org/licenses/LICENSE-2.0
8+
# Unless required by applicable law or agreed to in writing, software distributed
9+
# under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR
10+
# CONDITIONS OF ANY KIND, either express or implied. See the License for the
11+
# specific language governing permissions and limitations under the License.
12+
#
13+
# When you publish or redistribute any data created with VulnerableCode or any VulnerableCode
14+
# derivative work, you must accompany this data with the following acknowledgment:
15+
#
16+
# Generated with VulnerableCode and provided on an "AS IS" BASIS, WITHOUT WARRANTIES
17+
# OR CONDITIONS OF ANY KIND, either express or implied. No content created from
18+
# VulnerableCode should be considered or used as legal advice. Consult an Attorney
19+
# for any legal advice.
20+
# VulnerableCode is a free software code scanning tool from nexB Inc. and others.
21+
# Visit https://github.com/nexB/vulnerablecode/ for support and download.
22+
import json
23+
import logging
24+
from io import BytesIO
25+
from typing import Iterable
26+
from zipfile import ZipFile
27+
28+
import dateparser
29+
import requests
30+
from packageurl import PackageURL
31+
from univers.version_range import InvalidVersionRange
32+
from univers.version_range import PypiVersionRange
33+
from univers.versions import InvalidVersion
34+
from univers.versions import PypiVersion
35+
from univers.versions import SemverVersion
36+
37+
from vulnerabilities.helpers import dedupe
38+
from vulnerabilities.importer import AdvisoryData
39+
from vulnerabilities.importer import AffectedPackage
40+
from vulnerabilities.importer import Importer
41+
from vulnerabilities.importer import Reference
42+
from vulnerabilities.importer import VulnerabilitySeverity
43+
from vulnerabilities.severity_systems import SCORING_SYSTEMS
44+
45+
logger = logging.getLogger(__name__)
46+
47+
48+
class PyPIImporter(Importer):
49+
license_url = "https://github.com/pypa/advisory-database/blob/main/LICENSE"
50+
spdx_license_expression = "CC-BY-4.0"
51+
52+
def advisory_data(self) -> Iterable[AdvisoryData]:
53+
"""
54+
1. Fetch the data from osv api
55+
2. unzip the file
56+
3. open the file one by one
57+
4. yield the json file to parse_advisory_data
58+
"""
59+
url = "https://osv-vulnerabilities.storage.googleapis.com/PyPI/all.zip"
60+
response = requests.get(url).content
61+
with ZipFile(BytesIO(response)) as zip_file:
62+
for file_name in zip_file.namelist():
63+
if not file_name.startswith("PYSEC-"):
64+
logger.error(f"NotImplementedError PyPI package file_name: {file_name}")
65+
else:
66+
with zip_file.open(file_name) as f:
67+
vul_info = json.loads(f.read())
68+
yield parse_advisory_data(vul_info)
69+
70+
71+
def parse_advisory_data(raw_data: dict) -> AdvisoryData:
72+
summary = raw_data.get("summary") or ""
73+
aliases = get_aliases(raw_data)
74+
date_published = get_published_date(raw_data)
75+
severity = list(get_severities(raw_data))
76+
references = get_references(raw_data, severity)
77+
78+
affected_packages = []
79+
if "affected" not in raw_data:
80+
logger.error(f"affected_packages not found - {raw_data['id'] !r}")
81+
82+
if "affected" in raw_data:
83+
for affected_pkg in raw_data["affected"]:
84+
purl = get_affected_purl(affected_pkg, raw_data["id"])
85+
if purl.type != "pypi":
86+
logger.error(
87+
f"Non PyPI package found in PYSEC advisories: {purl} - from: {raw_data['id']!r}"
88+
)
89+
else:
90+
vulnid = raw_data["id"]
91+
affected_version_range = get_affected_version_range(affected_pkg, vulnid)
92+
for fixed_range in affected_pkg.get("ranges", []):
93+
fixed_version = get_fixed_version(fixed_range, vulnid)
94+
95+
for version in fixed_version:
96+
affected_packages.append(
97+
AffectedPackage(
98+
package=purl,
99+
affected_version_range=affected_version_range,
100+
fixed_version=version,
101+
)
102+
)
103+
104+
return AdvisoryData(
105+
aliases=aliases,
106+
summary=summary,
107+
affected_packages=affected_packages,
108+
references=references,
109+
date_published=date_published,
110+
)
111+
112+
113+
def fixed_filter(fixed_range) -> []:
114+
"""
115+
Return a list of fixed version strings given a ``fixed_range`` mapping of OSV data.
116+
>>> list(fixed_filter({"type": "SEMVER", "events": [{"introduced": "0"}, {"fixed": "1.6.0"}]}))
117+
['1.6.0']
118+
>>> list(fixed_filter({"type": "ECOSYSTEM","events":[{"introduced": "0"},{"fixed": "1.0.0"},{"fixed": "9.0.0"}]}))
119+
['1.0.0', '9.0.0']
120+
"""
121+
for event in fixed_range.get("events") or []:
122+
fixed = event.get("fixed")
123+
if fixed:
124+
yield fixed
125+
126+
127+
def get_aliases(raw_data) -> []:
128+
"""
129+
aliases field is optional , id is required and these are all aliases from our perspective
130+
converting list of two fields to a dict then , convert it to a list to make sure a list is unique
131+
>>> get_aliases({"id": "GHSA-j3f7-7rmc-6wqj"})
132+
['GHSA-j3f7-7rmc-6wqj']
133+
>>> get_aliases({"aliases": ["CVE-2021-40831"]})
134+
['CVE-2021-40831']
135+
>>> get_aliases({"aliases": ["CVE-2022-22817", "GHSA-8vj2-vxx3-667w"], "id": "GHSA-j3f7-7rmc-6wqj"})
136+
['CVE-2022-22817', 'GHSA-8vj2-vxx3-667w', 'GHSA-j3f7-7rmc-6wqj']
137+
"""
138+
vulnerability_id = raw_data.get("id")
139+
vulnerability_aliases = raw_data.get("aliases") or []
140+
if vulnerability_id:
141+
vulnerability_aliases.append(vulnerability_id)
142+
return vulnerability_aliases
143+
144+
145+
def get_published_date(raw_data):
146+
published = raw_data.get("published")
147+
return published and dateparser.parse(published)
148+
149+
150+
def get_severities(raw_data) -> []:
151+
if "severity" in raw_data:
152+
for sever_list in raw_data.get("severity") or []:
153+
if sever_list.get("type") == "CVSS_V3":
154+
yield VulnerabilitySeverity(
155+
system=SCORING_SYSTEMS["cvssv3.1_vector"], value=sever_list["score"]
156+
)
157+
else:
158+
logger.error(f"NotImplementedError severity type- {raw_data['id']!r}")
159+
160+
ecosys = raw_data.get("ecosystem_specific") or {}
161+
sever = ecosys.get("severity")
162+
if sever:
163+
yield VulnerabilitySeverity(
164+
system=SCORING_SYSTEMS["generic_textual"],
165+
value=sever,
166+
)
167+
168+
database_specific = raw_data.get("database_specific") or {}
169+
sever = database_specific.get("severity")
170+
if sever:
171+
yield VulnerabilitySeverity(
172+
system=SCORING_SYSTEMS["generic_textual"],
173+
value=sever,
174+
)
175+
176+
177+
def get_references(raw_data, severities) -> []:
178+
references = raw_data.get("references") or []
179+
return [Reference(url=ref["url"], severities=severities) for ref in references if ref]
180+
181+
182+
def get_affected_purl(affected_pkg, raw_id):
183+
package = affected_pkg.get("package") or {}
184+
purl = package.get("purl")
185+
if purl:
186+
try:
187+
return PackageURL.from_string(purl)
188+
except ValueError:
189+
logger.error(f"PackageURL ValueError - {raw_id !r} - purl: {purl !r}")
190+
191+
ecosys = package.get("ecosystem")
192+
name = package.get("name")
193+
if ecosys and name:
194+
return PackageURL(type=ecosys, name=name)
195+
else:
196+
logger.error(f"purl affected_pkg not found - {raw_id !r}")
197+
198+
199+
def get_affected_version_range(affected_pkg, raw_id):
200+
affected_versions = affected_pkg.get("versions")
201+
if affected_versions:
202+
try:
203+
return PypiVersionRange(affected_versions)
204+
except InvalidVersionRange:
205+
logger.error(f"InvalidVersionRange affected_pkg_version_range Error - {raw_id !r} ")
206+
else:
207+
logger.error(f"affected_pkg_version_range not found - {raw_id !r} ")
208+
209+
210+
def get_fixed_version(fixed_range, raw_id) -> []:
211+
"""
212+
Return a list of fixed versions, using fixed_filter we get the list of fixed version strings,
213+
then we pass every element to their univers.versions , then we dedupe the result
214+
>>> get_fixed_version({}, "GHSA-j3f7-7rmc-6wqj")
215+
[]
216+
>>> get_fixed_version({"type": "ECOSYSTEM", "events": [{"fixed": "1.7.0"}]}, "GHSA-j3f7-7rmc-6wqj")
217+
[PypiVersion(string='1.7.0')]
218+
"""
219+
fixed_version = []
220+
if "type" not in fixed_range:
221+
logger.error(f"Invalid type - {raw_id!r}")
222+
else:
223+
list_fixed = fixed_filter(fixed_range)
224+
fixed_range_type = fixed_range["type"]
225+
for i in list_fixed:
226+
if fixed_range_type == "ECOSYSTEM":
227+
try:
228+
fixed_version.append(PypiVersion(i))
229+
except InvalidVersion:
230+
logger.error(f"Invalid Version - PypiVersion - {raw_id !r} - {i !r}")
231+
if fixed_range_type == "SEMVER":
232+
try:
233+
fixed_version.append(SemverVersion(i))
234+
except InvalidVersion:
235+
logger.error(f"Invalid Version - SemverVersion - {raw_id !r} - {i !r}")
236+
if fixed_range_type == "GIT":
237+
# TODO add GitHubVersion univers fix_version
238+
logger.error(f"NotImplementedError GIT Version - {raw_id !r} - {i !r}")
239+
240+
return dedupe(fixed_version)

0 commit comments

Comments
 (0)