Skip to content

Upgrade go-git to v6 alpha.4, fixing GO-2026-5496 (external/trace snapshot)#49

Merged
Patel230 merged 4 commits into
mainfrom
feature/hawk-external-trace-20260705
Jul 8, 2026
Merged

Upgrade go-git to v6 alpha.4, fixing GO-2026-5496 (external/trace snapshot)#49
Patel230 merged 4 commits into
mainfrom
feature/hawk-external-trace-20260705

Conversation

@Patel230

@Patel230 Patel230 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

This branch is off an older, diverged checkout of trace used as hawk's external/trace git submodule (pinned at 38a3993). The same fix has already been applied to the up-to-date trace repo in PR #48; this PR brings the security fix to this older submodule snapshot until it can be re-synced to trace's current main.

  • Upgrade go-git v6 to alpha.4, fixing GO-2026-5496 (SSH transport single-quote escaping)
  • Migrate all call sites to the new Worktree.Filesystem() method API
  • Fix zero-hash tree entry rejection in v2_generation_test.go

Test plan

  • GOWORK=off go build ./...
  • GOWORK=off go vet ./...
  • GOWORK=off go test ./...

Patel230 added 4 commits July 7, 2026 10:17
- Integer-overflow guards on numeric conversions
- File/dir permission tightening (0600/0750)
- Path-traversal cleaning and error-return handling
- Narrow, justified #nosec annotations where risk is not applicable

Module now scans clean with gosec (0 issues).
- Migrate all call sites to the new Worktree.Filesystem() method API
- Fix zero-hash tree entry rejection in v2_generation_test.go
@Patel230 Patel230 merged commit 7006b21 into main Jul 8, 2026
7 of 8 checks passed
@Patel230 Patel230 deleted the feature/hawk-external-trace-20260705 branch July 8, 2026 17:43
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