Skip to content

security(trivy): add trivy to supply chain#793

Merged
moscaale merged 3 commits intomainfrom
661-trivy-scan
Mar 25, 2026
Merged

security(trivy): add trivy to supply chain#793
moscaale merged 3 commits intomainfrom
661-trivy-scan

Conversation

@moscaale
Copy link
Copy Markdown
Contributor

Description

Please provide a summarized description of what was done in this PR.

Precise the issue that you are resolving.

Overview

This section provides a checklist to help categorize and describe the changes made in this PR.

Area

Please select the area(s) that this PR affects:

  • Core / Global project settings
  • API
  • Playground / Web UI
  • DevOps
  • Docusaurus / Documentation
  • Other (specify below)

If "Other" is selected, please provide more details about the area(s) affected by this PR here, otherwise delete this part.

Type of change

Please select the type of change that this PR introduces:

  • New feature
  • Bugfix
  • Enhancement (improvement of an existing feature)
  • Refactor (change that neither fixes a bug nor modifies behavior)
  • Documentation
  • Tests
  • Performance improvement
  • Chore / Maintenance (change to the build process or auxiliary tools, dependencies update, etc.)

Definition of Done / Technical changes

Please provide the Definition of Done (DoD) criteria that apply to this PR.

  • DoD 1
  • DoD 2
  • DoD 3
  • ...

Screenshots / Demo (if applicable)

Please, attach screenshots or a link to a demo / video demonstrating the changes made in this PR.

Breaking changes

Please select one of the following options:

  • No breaking changes
  • This PR contains breaking changes (explain below)

Please describe the breaking change introduced by this PR here, otherwise delete this part.

NB: A breaking change is a modification that is not backwards-compatible and/or changes current functionality.

Quality assurance & Review readiness

Before requesting a review, please take a moment to confirm that the following aspects have been considered and addressed.

This section helps ensure the PR is ready for review, safe to merge, and deployable. If any items are left unchecked, please add a brief explanation for context.

Documentation

Please select one of the following options:

  • No documentation needed
  • README / Markdown files updated
  • API documentation updated (Swagger / Redoc)
  • Docstrings updated
  • Inline code comments added where needed

Tests

Please select one or more of the following options:

  • No tests added (explain below)
  • Unit tests added
  • Integration tests added
  • Functional tests added
  • End-to-end tests added
  • Performance tests added
  • Existing tests updated

If no tests were added, please explain why here, otherwise delete this part.

NB: For a concise overview of software testing types, see this Atlassian's guide.

Code Standards

  • Code follows project conventions and architecture
  • No unused imports, variables, functions, or classes
  • No debug logs or commented-out code left
  • No secrets or environment variables committed in clear text
  • Code is linted and formatted using the project tools (ruff, etc.)
  • N/A

Git & Process Standards

  • PR title follows Conventional Commits
  • PR is correctly labeled
  • PR is linked to relevant issue(s) / project(s)
  • Author is assigned to the PR
  • At least one reviewer has been requested

Deployment Notes

  • No special deployment steps required
  • Requires database migration (see "Database migration" section)
  • Requires new or updated environment variables (explain below)
  • Requires other special deployment steps (explain below)

If new or updated environment variables are required, please list them here, otherwise delete this part.
If other special deployment steps are required, please describe them here, otherwise delete this part.

Database migration

Please select one of the following options:

  • No database migration required
  • This PR requires a database migration (see checklist below)

Please confirm that the following steps have been completed for the database migration:

  • Migration script added to api/alembic/versions/ folder
  • Migration upgrade tested locally
  • Migration downgrade tested locally
  • Migration documented (if applicable)

Reviewer Focus

Please provide any specific areas you would like the reviewers to focus on during their review of this PR (complex logic, risky changes, performance-sensitive code, etc.).

Additional Notes

Please provide any additional information or context that may be relevant to this PR, otherwise delete this part.

@moscaale moscaale self-assigned this Mar 23, 2026
@moscaale moscaale changed the title security(tricy): add trivy to supple chain security(trivy): add trivy to supple chain Mar 24, 2026
@moscaale moscaale force-pushed the 661-trivy-scan branch 2 times, most recently from cbe1e18 to 38014c0 Compare March 24, 2026 09:54
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@moscaale moscaale force-pushed the 661-trivy-scan branch 13 times, most recently from 11aa6b2 to 7bfab4e Compare March 24, 2026 13:51
@moscaale moscaale changed the title security(trivy): add trivy to supple chain security(trivy): add trivy to supply chain Mar 24, 2026
@moscaale moscaale force-pushed the 661-trivy-scan branch 5 times, most recently from 809e941 to fbc62f5 Compare March 24, 2026 14:39
@moscaale moscaale merged commit 655a8b1 into main Mar 25, 2026
2 checks passed
@moscaale moscaale deleted the 661-trivy-scan branch March 25, 2026 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants