Skip to content

chore(security): add ReversingLabs malware scan workflow - #457

Merged
developerkunal merged 3 commits into
masterfrom
security/add-malwarescan
Oct 19, 2026
Merged

developerkunal merged 3 commits into
masterfrom
security/add-malwarescan

Conversation

@nirmal-joishi-a0

@nirmal-joishi-a0 nirmal-joishi-a0 commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

🔧 Changes

Adds a ReversingLabs malware scan to the release pipeline.

  • .github/workflows/rl.yml is a reusable workflow. It packages the tracked source as go-jwt-middleware-<version>.tar.gz with git archive, reading the version from .version, and scans that file with the rl-scan action from auth0/devsecops-tooling. The job fails unless the scan status is success.
  • .github/workflows/release.yml gets a new rl-scan job that calls rl.yml on the same trigger as the release job (a merged release/* PR, or a manual dispatch). The release job does not depend on it, so a failed scan fails its own job but does not block the release.

The scan needs the eight RL and ProdSec org secrets to be available to this repository, and the AWS role behind PRODSEC_TOOLS_ARN must trust it.

🔬 Testing

This has not run end to end yet, because it needs the org secrets and the AWS trust for this repository. It runs on the next release or a manual dispatch of the release workflow.

@nirmal-joishi-a0
nirmal-joishi-a0 requested a review from a team as a code owner October 12, 2026 15:13
@nirmal-joishi-a0

Copy link
Copy Markdown
Contributor Author

@auth0/project-dx-sdks-engineer-codeowner please review the files in the PR. This automated security-hardening workflow is untested in this repo — before merging, confirm it triggers and passes (and is not silently ignoring failures); do not merge on a green result alone.

@nirmal-joishi-a0

Copy link
Copy Markdown
Contributor Author

An internal service ticket has been filed for the owning team to review and merge this PR.

@codecov-commenter

codecov-commenter commented Oct 12, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.52%. Comparing base (cdf2ba7) to head (150f908).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #457   +/-   ##
=======================================
  Coverage   94.52%   94.52%           
=======================================
  Files          25       25           
  Lines        2210     2210           
=======================================
  Hits         2089     2089           
  Misses         82       82           
  Partials       39       39           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Package the tracked source as a tar.gz in the reusable rl.yml workflow and scan it with the rl-scan action. Call it from release.yml alongside the release job. A failed scan fails its own run but does not block the release.
@developerkunal
developerkunal merged commit 72db019 into master Oct 19, 2026
13 checks passed
@developerkunal
developerkunal deleted the security/add-malwarescan branch October 19, 2026 13:51
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.

4 participants