Skip to content

ci: pin golangci-lint to v2.6.0 (fix latest-drift gosec G703/G704)#25

Merged
kamir merged 1 commit into
mainfrom
ci/pin-golangci-lint-v2
Jun 17, 2026
Merged

ci: pin golangci-lint to v2.6.0 (fix latest-drift gosec G703/G704)#25
kamir merged 1 commit into
mainfrom
ci/pin-golangci-lint-v2

Conversation

@kamir

@kamir kamir commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #24 to make CI deterministic. CI used golangci-lint version: latest, which drifted to a newer release whose gosec G703 (path-traversal) and G704 (SSRF) taint rules fail on patterns kshark guards at runtime (file reads in diff.go/util.go; the SSRF-checked httpGet in validate_blueprint.go). #24 validated v2.6.0 locally and it passed; this pins CI to v2.6.0 so CI matches local. Upgrade deliberately and re-validate. Only the golangci-lint-action version is pinned; goreleaser stays on latest.

Follow-up to #24. CI used golangci-lint version: latest, which drifted to a newer
release whose gosec G703 (path-traversal) + G704 (SSRF) taint rules fail on patterns
kshark guards at runtime (file reads in diff.go/util.go; the SSRF-checked httpGet in
validate_blueprint.go). #24 validated v2.6.0 locally as green; pin CI to v2.6.0 so it
matches. Bump deliberately + re-validate. Only golangci-lint is pinned; goreleaser stays latest.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@kamir kamir merged commit d1ad06e into main Jun 17, 2026
2 checks passed
@kamir kamir deleted the ci/pin-golangci-lint-v2 branch June 17, 2026 14:27
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.

1 participant