Skip to content

Add Snyk Elixir security scan workflow - #5

Merged
mikhailbahdashych-glia merged 2 commits into
masterfrom
add-snyk-elixir-security-scan
Aug 19, 2026
Merged

Add Snyk Elixir security scan workflow#5
mikhailbahdashych-glia merged 2 commits into
masterfrom
add-snyk-elixir-security-scan

Conversation

@mikhailbahdashych-glia

Copy link
Copy Markdown
Contributor

Why

This repository is categorised production_library in DevHub, which requires Snyk coverage. Elixir has no Snyk SCM support — importing through the GitHub Cloud App produces no projects — so the CLI workflow is the only path that scans mix.lock (hex dependencies).

Docs: Snyk for Elixir projects

What this does

  • pull_request / merge_group: tests the dependency tree the change would introduce
  • push to master: (re-)registers the repo's hex projects in Snyk org glia-sudo (via the SNYK_ORGANIZATION_SLUG Actions variable, already set on this repository)
  • weekly schedule + workflow_dispatch: keeps the snapshot fresh when the repository is quiet

Merging this PR is what creates the Snyk projects — the push run performs the snyk monitor registration.

🤖 Generated with Claude Code

Registers and keeps fresh the hex dependency scan via the shared
elixir-snyk-security-scan workflow, per the security documentation.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mikhailbahdashych-glia
mikhailbahdashych-glia requested a review from a team as a code owner August 18, 2026 09:03
@mikhailbahdashych-glia
mikhailbahdashych-glia requested review from KristinaNa and removed request for a team August 18, 2026 09:03
@mikhailbahdashych-glia
mikhailbahdashych-glia marked this pull request as draft August 18, 2026 09:19
@mikhailbahdashych-glia

Copy link
Copy Markdown
Contributor Author

⚠️ Converted to draft — do not merge yet. This repository is public, and the workflow as written cannot run here for two reasons: (1) public repos cannot call the private salemove/glia-security-workflows reusable workflow, and (2) the org-level SNYK_TOKEN secret is not shared with public repositories. A revised, self-contained version will follow once the approach is agreed. (Found by post-creation audit.)

Public repositories cannot call the private glia-security-workflows
reusable workflow, so the scan steps are inlined here directly, with the
standard triggers (schedule, workflow_dispatch) from the security docs.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@mikhailbahdashych-glia

Copy link
Copy Markdown
Contributor Author

✅ Reworked per the security team's decision: the workflow is now self-contained — the scan steps from the shared glia-security-workflows reusable workflow are inlined directly, since a public repository cannot call a private reusable workflow. Triggers follow the security docs (weekly schedule + manual dispatch).

Remaining blocker before un-drafting: a SNYK_TOKEN secret available to this public repository — none is visible yet. The snyk monitor step is additionally gated on the SNYK_ORGANIZATION_SLUG Actions variable (already set), so merging can never register into a wrong Snyk org.

@mikhailbahdashych-glia
mikhailbahdashych-glia marked this pull request as ready for review August 18, 2026 13:01
@mikhailbahdashych-glia
mikhailbahdashych-glia merged commit e6986d9 into master Aug 19, 2026
6 of 7 checks passed
@mikhailbahdashych-glia
mikhailbahdashych-glia deleted the add-snyk-elixir-security-scan branch August 19, 2026 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants