Skip to content

build: harden CI, split docker workflow, update dependencies#72

Merged
umputun merged 1 commit intomasterfrom
worktree-ci-hardening
Mar 29, 2026
Merged

build: harden CI, split docker workflow, update dependencies#72
umputun merged 1 commit intomasterfrom
worktree-ci-hardening

Conversation

@paskal
Copy link
Copy Markdown
Member

@paskal paskal commented Mar 29, 2026

Summary

  • Split docker build into separate workflow (docker.yml) with workflow_run trigger, decoupling it from CI
  • Harden against CI injection attacks: env-var indirection for all ${{ }} expressions in run: blocks, event == 'push' guard on workflow_run trigger
  • Replace unmaintained wbari/start-mongoDB action with native service container
  • Update action versions: docker/login-action v3→v4, docker/build-push-action v6→v7, actions/upload-artifact v4→v7, actions/download-artifact v4→v8
  • Fix dependabot gomod directory /backend/ (broken since project structure flatten)
  • Update all Go dependencies; removes vulnerable go.opentelemetry.io/otel/sdk (CVE-2026-24051)
  • Apply go fix: use builtin min(), range-over-int loops

- Split docker build into separate workflow with workflow_run trigger
- Harden against CI injection: env-var indirection for all expressions in
  run blocks, event=='push' guard on workflow_run trigger
- Replace unmaintained wbari/start-mongoDB action with native service container
- Update action versions: login v3→v4, build-push v6→v7,
  upload-artifact v4→v7, download-artifact v4→v8
- Fix dependabot gomod directory /backend → / (post structure flatten)
- Update all Go dependencies, removing vulnerable otel/sdk (CVE-2026-24051)
- Apply go fix: use builtin min(), range-over-int loops
@paskal paskal force-pushed the worktree-ci-hardening branch from 6a92058 to f214eb6 Compare March 29, 2026 09:50
@umputun umputun merged commit e8a8df1 into master Mar 29, 2026
2 checks passed
@umputun umputun deleted the worktree-ci-hardening branch March 29, 2026 19:14
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