Skip to content

Scorecard Integration#1294

Closed
404-geek wants to merge 76 commits into
aboutcode-org:mainfrom
404-geek:scorecard_integration
Closed

Scorecard Integration#1294
404-geek wants to merge 76 commits into
aboutcode-org:mainfrom
404-geek:scorecard_integration

Conversation

@404-geek

@404-geek 404-geek commented Jun 26, 2024

Copy link
Copy Markdown
Collaborator

ScoreCode Integration

This pull request integrates the ScoreCode Repo into SCIO, enabling the fetching of the latest OSSF Scorecard Data for discovered packages using their vcs_url. The current implementation supports github.com and gitlab.com VCS URLs.

Key Features:

  • Integration with ScoreCode Repo
  • Fetching of OSSF Scorecard Data using vcs_url
  • Support for github.com and gitlab.com VCS URLs

Related Issues:

This feature enhances SCIO's functionality by ensuring that users can retrieve the most up-to-date security scores for packages discovered in their projects, improving overall security assessment and management.

404-geek added 30 commits June 26, 2024 13:20
developed functions to check for availability nexB#598

Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
…aving logic nexB#1283

Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
…exB#1283

Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
… nexB#598

Signed-off-by: 404-geek <pranayd61@gmail.com>
…up.cfg nexB#1283

Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
…gration

# Conflicts:
#	scanpipe/models.py
#	scanpipe/tests/test_models.py
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>

@tdruez tdruez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@404-geek See my various comments ;)

Also, the new pipeline needs to be added to the built-in-pipelines.rst documentation.

Comment thread scanpipe/migrations/0070_alter_project_purl_discoveredpackagescore_and_more.py Outdated
Comment thread scanpipe/models.py Outdated
Comment thread scanpipe/models.py Outdated
)

@classmethod
@transaction.atomic()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@404-geek You haven't address the question above yet ;)

Comment thread scanpipe/models.py Outdated
Comment thread scanpipe/models.py Outdated
Comment thread scanpipe/pipelines/fetch_scorecode_info.py Outdated
Comment thread scanpipe/tests/__init__.py Outdated
Comment thread scanpipe/tests/__init__.py Outdated
Comment thread scanpipe/tests/test_models.py Outdated
Comment thread scanpipe/tests/test_pipelines.py Outdated
@404-geek 404-geek requested a review from tdruez March 2, 2025 06:46

@tdruez tdruez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would also suggest to simplify the pipeline and module names:

  • fetch_scorecode_info -> fetch_scores
  • FetchScoreCodeInfo -> FetchScores

Comment thread setup.cfg Outdated
Comment thread scanpipe/tests/test_pipelines.py Outdated
Comment thread scanpipe/tests/test_pipelines.py Outdated
Comment thread scanpipe/tests/test_pipelines.py
Comment thread scanpipe/tests/regen_test_data.py Outdated
Comment thread scanpipe/models.py Outdated
Comment thread scanpipe/models.py
Comment thread scanpipe/tests/test_models.py
Comment thread scanpipe/tests/test_models.py Outdated
Comment thread scanpipe/tests/test_models.py Outdated
404-geek added 6 commits July 1, 2025 00:28
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
@404-geek 404-geek requested a review from tdruez July 14, 2025 13:44
Comment thread scanpipe/migrations/0074_alter_codebaseresource_sha1_git_and_more.py Outdated
@tdruez

tdruez commented Jul 14, 2025

Copy link
Copy Markdown
Contributor

@404-geek We are almost ready but there's still a few comments you have not addressed yet.

404-geek added 2 commits July 17, 2025 02:03
Signed-off-by: 404-geek <pranayd61@gmail.com>
Signed-off-by: 404-geek <pranayd61@gmail.com>
@404-geek 404-geek requested a review from tdruez July 17, 2025 13:08
@tdruez

tdruez commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

Thanks @404-geek for the last push on this.
Merging the branch with a few adjusments at #1777
Great work! 👍

@tdruez tdruez closed this Jul 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enrich an SBOM using OSSF Security Score Card Store OSSF scorecard data in scancode.io models

4 participants