-
Notifications
You must be signed in to change notification settings - Fork 109
Scorecard Integration #1294
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Scorecard Integration #1294
Changes from 31 commits
173db43
272b99c
944cee2
bc445c1
f241b3b
605a5cf
3833dca
37380c8
5502b5f
2aeb2a4
103fca0
952e6a6
3ba3db5
2f2b846
39a056d
e5f3e7a
c2f5c4d
6ee5b07
760afc2
0dbc92f
d652f42
aa154c0
37ad73a
563991b
4632dfc
923c834
94bfcd0
d129f73
24c7be0
259f004
ccd75ae
9d80ef1
29da290
9d72734
301122e
5812d97
df50416
88a43dc
fc4945e
c00b3b3
b97ff7a
496945b
f86d5bb
36e955a
90c113a
43886a3
7c134c7
3d4d6ea
f4ed4b5
b9229a5
f80f111
ee2ea14
bd5b9b3
2b4629e
99ee48d
80153f8
77efae0
113a557
64e17fe
027fe2d
7f43f9e
89d2636
f6f19a7
e2eaadf
3baeac3
601201b
c6b9832
8679ddc
8e7fd43
c6a8ac5
7cd0706
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,151 @@ | ||
# Generated by Django 5.0.6 on 2024-07-11 18:26 | ||
|
||
import django.db.models.deletion | ||
import uuid | ||
from django.db import migrations, models | ||
|
||
|
||
class Migration(migrations.Migration): | ||
|
||
dependencies = [ | ||
("scanpipe", "0063_run_selected_steps"), | ||
] | ||
|
||
operations = [ | ||
migrations.CreateModel( | ||
name="PackageScore", | ||
fields=[ | ||
( | ||
"scoring_tool", | ||
models.CharField( | ||
blank=True, | ||
choices=[("ossf-scorecard", "Ossf"), ("others", "Others")], | ||
help_text="Defines the source of a score or any other scoring metricsFor example: ossf-scorecard for scorecard data", | ||
max_length=100, | ||
), | ||
), | ||
( | ||
"scoring_tool_version", | ||
models.CharField( | ||
blank=True, | ||
help_text="Defines the version of the scoring tool used for scanning thepackageFor Eg : 4.6 current version of OSSF - scorecard", | ||
max_length=50, | ||
), | ||
), | ||
( | ||
"score", | ||
models.CharField( | ||
blank=True, | ||
help_text="Score of the package which is scanned", | ||
max_length=50, | ||
), | ||
), | ||
( | ||
"scoring_tool_documentation_url", | ||
models.CharField( | ||
blank=True, | ||
help_text="Documentation URL of the scoring tool used", | ||
max_length=100, | ||
), | ||
), | ||
( | ||
"score_date", | ||
models.DateTimeField( | ||
blank=True, | ||
editable=False, | ||
help_text="Date when the scoring was calculated on the package", | ||
null=True, | ||
), | ||
), | ||
( | ||
"uuid", | ||
models.UUIDField( | ||
db_index=True, | ||
default=uuid.uuid4, | ||
editable=False, | ||
primary_key=True, | ||
serialize=False, | ||
verbose_name="UUID", | ||
), | ||
), | ||
( | ||
"discovered_package", | ||
models.ForeignKey( | ||
blank=True, | ||
editable=False, | ||
help_text="The package for which the score is given", | ||
null=True, | ||
on_delete=django.db.models.deletion.CASCADE, | ||
related_name="discovered_packages_score", | ||
to="scanpipe.discoveredpackage", | ||
), | ||
), | ||
], | ||
options={ | ||
"abstract": False, | ||
}, | ||
), | ||
migrations.CreateModel( | ||
name="ScorecardCheck", | ||
fields=[ | ||
( | ||
"check_name", | ||
models.CharField( | ||
blank=True, | ||
help_text="Defines the name of check corresponding to the OSSF scoreFor example: Code-Review or CII-Best-PracticesThese are the some of the checks which are performed on a scanned package", | ||
max_length=100, | ||
), | ||
), | ||
( | ||
"check_score", | ||
models.CharField( | ||
blank=True, | ||
help_text="Defines the score of the check for the package scannedFor Eg : 9 is a score given for Code-Review", | ||
max_length=50, | ||
), | ||
), | ||
( | ||
"reason", | ||
models.CharField( | ||
blank=True, | ||
help_text="Gives a reason why a score was given for a specific checkFor eg, : Found 9/10 approved changesets -- score normalized to 9", | ||
max_length=300, | ||
), | ||
), | ||
( | ||
"details", | ||
models.JSONField( | ||
blank=True, | ||
default=list, | ||
help_text="A list of details/errors regarding the score", | ||
), | ||
), | ||
( | ||
"uuid", | ||
models.UUIDField( | ||
db_index=True, | ||
default=uuid.uuid4, | ||
editable=False, | ||
primary_key=True, | ||
serialize=False, | ||
verbose_name="UUID", | ||
), | ||
), | ||
( | ||
"for_package_score", | ||
models.ForeignKey( | ||
blank=True, | ||
editable=False, | ||
help_text="The checks for which the score is given", | ||
null=True, | ||
on_delete=django.db.models.deletion.CASCADE, | ||
related_name="discovered_packages_score_checks", | ||
to="scanpipe.packagescore", | ||
), | ||
), | ||
], | ||
options={ | ||
"abstract": False, | ||
}, | ||
), | ||
] |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -29,6 +29,7 @@ | |
from collections import Counter | ||
from collections import defaultdict | ||
from contextlib import suppress | ||
from datetime import datetime | ||
from itertools import groupby | ||
from operator import itemgetter | ||
from pathlib import Path | ||
|
@@ -73,6 +74,8 @@ | |
from licensedcode.cache import build_spdx_license_expression | ||
from licensedcode.cache import get_licensing | ||
from matchcode_toolkit.fingerprinting import IGNORED_DIRECTORY_FINGERPRINTS | ||
from ossf_scorecard.contrib.django.models import PackageScoreMixin | ||
from ossf_scorecard.contrib.django.models import ScorecardChecksMixin | ||
from packagedcode.models import build_package_uid | ||
from packagedcode.utils import get_base_purl | ||
from packageurl import PackageURL | ||
|
@@ -1772,10 +1775,10 @@ class Run(UUIDPKModel, ProjectRelatedModel, AbstractTaskFieldsModel): | |
scancodeio_version = models.CharField(max_length=100, blank=True) | ||
description = models.TextField(blank=True) | ||
current_step = models.CharField(max_length=256, blank=True) | ||
selected_groups = models.JSONField( | ||
selected_steps = models.JSONField( | ||
null=True, blank=True, validators=[validate_none_or_list] | ||
) | ||
selected_steps = models.JSONField( | ||
selected_groups = models.JSONField( | ||
tdruez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
null=True, blank=True, validators=[validate_none_or_list] | ||
) | ||
|
||
|
@@ -3777,6 +3780,99 @@ def as_spdx(self): | |
) | ||
|
||
|
||
class PackageScore(UUIDPKModel, PackageScoreMixin): | ||
|
||
def __str__(self): | ||
return self.score or str(self.uuid) | ||
|
||
404-geek marked this conversation as resolved.
Show resolved
Hide resolved
|
||
discovered_package = models.ForeignKey( | ||
DiscoveredPackage, | ||
related_name="discovered_packages_score", | ||
help_text=_("The package for which the score is given"), | ||
on_delete=models.CASCADE, | ||
editable=False, | ||
blank=True, | ||
null=True, | ||
404-geek marked this conversation as resolved.
Show resolved
Hide resolved
|
||
) | ||
|
||
@classmethod | ||
@transaction.atomic() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure why this is required here? We are not doing multiple database updates that could crash and needs to be handled? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since we are updating the checks and score table at one go for that reason, I have kept it atomic There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @404-geek Could you provide an example that shows why atomic() is useful here? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @404-geek You haven't address the question above yet ;) |
||
def create_from_data(cls, DiscoveredPackage, scorecard_data, scoring_tool=None): | ||
tdruez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
"""Create ScoreCard Object from ScoreCard Object""" | ||
tdruez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
final_data = { | ||
"score": scorecard_data.score, | ||
"scoring_tool_version": scorecard_data.scoring_tool_version, | ||
"scoring_tool_documentation_url": ( | ||
scorecard_data.scoring_tool_documentation_url | ||
), | ||
} | ||
|
||
date_str = scorecard_data.score_date | ||
if date_str: | ||
|
||
naive_datetime = datetime.strptime(date_str, "%Y-%m-%d") | ||
|
||
score_date = timezone.make_aware( | ||
naive_datetime, timezone.get_current_timezone() | ||
) | ||
else: | ||
score_date = timezone.now() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is there cases where we don't have dates? Can you show some examples? |
||
|
||
final_data["score_date"] = score_date | ||
|
||
scorecard_object = cls.objects.create( | ||
**final_data, | ||
discovered_package=DiscoveredPackage, | ||
scoring_tool=scoring_tool, | ||
) | ||
|
||
# Create associated scorecard_checks | ||
checks_data = scorecard_data.checks | ||
|
||
ScorecardCheck.objects.bulk_create( | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are there so many checks where this would make a difference in time taken to save these? |
||
[ | ||
ScorecardCheck( | ||
check_name=check_data.check_name, | ||
check_score=check_data.check_score, | ||
reason=check_data.reason or "", | ||
details=check_data.details or [], | ||
for_package_score=scorecard_object, | ||
) | ||
for check_data in checks_data | ||
] | ||
) | ||
|
||
return scorecard_object | ||
|
||
|
||
class ScorecardCheck(UUIDPKModel, ScorecardChecksMixin): | ||
|
||
def __str__(self): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is this required? Do you show this anywhere? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I just kept it like that Do you have anything in mind to return if someone calls the instance directly.
404-geek marked this conversation as resolved.
Show resolved
Hide resolved
|
||
return self.check_score or str(self.uuid) | ||
|
||
for_package_score = models.ForeignKey( | ||
PackageScore, | ||
related_name="discovered_packages_score_checks", | ||
help_text=_("The checks for which the score is given"), | ||
on_delete=models.CASCADE, | ||
editable=False, | ||
blank=True, | ||
null=True, | ||
404-geek marked this conversation as resolved.
Show resolved
Hide resolved
|
||
) | ||
|
||
@classmethod | ||
def create_from_data(cls, package_score, check_data): | ||
"""Create a ScorecardCheck instance from provided data.""" | ||
final_data = { | ||
"check_name": check_data.get("name"), | ||
"check_score": check_data.get("score"), | ||
"reason": check_data.get("reason"), | ||
"details": check_data.get("details", []), | ||
tdruez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
"for_package_score": package_score, | ||
} | ||
return cls.objects.create(**final_data) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why not create in directly and have defaults? Doesn't seem like we are doing much here There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @AyanSinhaMahapatra thanks for pointing out. |
||
|
||
|
||
def normalize_package_url_data(purl_mapping, ignore_nulls=False): | ||
""" | ||
Normalize a mapping of purl data so database queries with | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
# SPDX-License-Identifier: Apache-2.0 | ||
# | ||
# http://nexb.com and https://github.com/nexB/scancode.io | ||
# The ScanCode.io software is licensed under the Apache License version 2.0. | ||
# Data generated with ScanCode.io is provided as-is without warranties. | ||
# ScanCode is a trademark of nexB Inc. | ||
# | ||
# You may not use this software except in compliance with the License. | ||
# You may obtain a copy of the License at: http://apache.org/licenses/LICENSE-2.0 | ||
# Unless required by applicable law or agreed to in writing, software distributed | ||
# under the License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR | ||
# CONDITIONS OF ANY KIND, either express or implied. See the License for the | ||
# specific language governing permissions and limitations under the License. | ||
# | ||
# Data Generated with ScanCode.io is provided on an "AS IS" BASIS, WITHOUT WARRANTIES | ||
# OR CONDITIONS OF ANY KIND, either express or implied. No content created from | ||
# ScanCode.io should be considered or used as legal advice. Consult an Attorney | ||
# for any legal advice. | ||
# | ||
# ScanCode.io is a free software code scanning tool from nexB Inc. and others. | ||
# Visit https://github.com/nexB/scancode.io for support and download. | ||
|
||
from ossf_scorecard import scorecard | ||
|
||
from scanpipe.pipelines import Pipeline | ||
|
||
|
||
class FetchScoreCodeInfo(Pipeline): | ||
tdruez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
""" | ||
Fetch scorecode information for packages and dependencies. | ||
|
||
scorecode data is stored on each package and dependency instance. | ||
AyanSinhaMahapatra marked this conversation as resolved.
Show resolved
Hide resolved
|
||
""" | ||
|
||
download_inputs = False | ||
is_addon = True | ||
|
||
@classmethod | ||
def steps(cls): | ||
return ( | ||
cls.check_scorecode_service_availability, | ||
cls.lookup_packages_scorecode_info, | ||
# cls.lookup_dependencies_scorecode_info, | ||
AyanSinhaMahapatra marked this conversation as resolved.
Show resolved
Hide resolved
|
||
) | ||
|
||
def check_scorecode_service_availability(self): | ||
"""Check if the scorecode service is configured and available.""" | ||
if not scorecard.is_configured(): | ||
raise Exception("scorecode service is not configured.") | ||
|
||
if not scorecard.is_available(): | ||
raise Exception("scorecode service is not available.") | ||
|
||
def lookup_packages_scorecode_info(self): | ||
"""Fetch scorecode information for each of the project's discovered packages.""" | ||
packages = self.project.discoveredpackages.all() | ||
scorecard.fetch_scorecard_info( | ||
packages=packages, | ||
logger=self.log, | ||
) |
Uh oh!
There was an error while loading. Please reload this page.