Skip to content

Potential fix for code scanning alert no. 54: Use of a known vulnerable action #242

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

Merged
merged 2 commits into from
May 29, 2025

Conversation

habib-deriv
Copy link
Contributor

Potential fix for https://github.com/deriv-com/deriv-api-docs/security/code-scanning/54

To fix the issue, the workflow should update the actions/download-artifact action to version 4.1.3, which is the latest secure version as recommended. This involves replacing the specific commit hash (6b208ae046db98c579e8a3aa621ab581ff575935) with the version tag v4.1.3. This change ensures that the workflow uses a secure and patched version of the action while maintaining its functionality.


Suggested fixes powered by Copilot Autofix. Review carefully before merging.

…le action

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Copy link

vercel bot commented May 29, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-api-docs ✅ Ready (Inspect) Visit Preview May 29, 2025 5:02am

Copy link

github-actions bot commented May 29, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/download-artifact d3f86a106a0bac45b974a628896c90dbdf5c8093 🟢 5.7
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 3 issue activity found in the last 90 days -- score normalized to 10
Code-Review🟢 10all changesets reviewed
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Pinned-Dependencies⚠️ 1dependency not pinned by hash detected -- score normalized to 1
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
SAST🟢 10SAST tool is run on all commits
Vulnerabilities⚠️ 19 existing vulnerabilities detected
actions/actions/download-artifact 6b208ae046db98c579e8a3aa621ab581ff575935 🟢 5.7
Details
CheckScoreReason
Maintained🟢 1030 commit(s) and 3 issue activity found in the last 90 days -- score normalized to 10
Code-Review🟢 10all changesets reviewed
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Packaging⚠️ -1packaging workflow not detected
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Pinned-Dependencies⚠️ 1dependency not pinned by hash detected -- score normalized to 1
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
SAST🟢 10SAST tool is run on all commits
Vulnerabilities⚠️ 19 existing vulnerabilities detected

Scanned Manifest Files

.github/workflows/release_production.yml

@habib-deriv habib-deriv marked this pull request as ready for review May 29, 2025 05:00
@habib-deriv habib-deriv requested a review from shafin-deriv as a code owner May 29, 2025 05:00
@mayuran-deriv mayuran-deriv merged commit db5ee35 into master May 29, 2025
6 checks passed
@mayuran-deriv mayuran-deriv deleted the alert-autofix-54 branch May 29, 2025 06:40
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.

2 participants