mirror of
https://github.com/safedep/pmg.git
synced 2026-08-03 07:24:09 +02:00
* feat: add experimental Go module support via pmg go Adds Go modules as a proxy-guarded ecosystem, opt-in only: the command runs solely when invoked explicitly as `pmg go ...` and is deliberately excluded from setup aliases and PATH shims so existing users are unaffected. - packagemanager: goPackageManager with fail-safe command classification (vet/fix excluded from non-download since they can fetch on a cold cache) and pinned-version extraction where only canonical semver counts as explicit. - GOPROXY normalization (fail-closed): effective GOPROXY read via `go env` (honors go env -w), rebuilt comma-joined with `direct` dropped so a 403 block is terminal and nothing silently falls back to unanalyzed VCS fetches. GOPRIVATE/GONOPROXY surface a warning; GOINSECURE is cleared. Contributed to the proxy flow through a new ProxyRoutingProvider hook (extra child env + dynamic MITM hosts). - Go interceptor with dynamic host matching from the user's effective GOPROXY via InterceptorContext.GoProxyHosts. Malware analysis runs on .zip only (the sole endpoint that delivers code); .info/.mod/@latest/ list pass through; /sumdb/ traffic and sum.golang.org are never touched so checksum-db verification stays intact; golang.org/toolchain is allowed on Go's own checksum verification. - Dependency cooldown: publish time captured from .info responses (body unmodified), in-window .zip blocked with 403; fails open for cooldown only when the publish time was never observed. - Cert gate: on macOS/Windows `pmg go` fails fast with actionable guidance unless the persisted PMG CA is OS-trusted (Go ignores SSL_CERT_FILE there); Linux works via the injected bundle. - proxye2e: GOPROXY-protocol mock registry, Go driver and 10 hermetic cases (allow/block/confirm, case-escaped paths, cooldown block and fail-open, toolchain, sumdb passthrough). Verified end-to-end on Linux: `pmg go get github.com/google/uuid@v1.6.0` MITMs proxy.golang.org, analyzes the decoded module at the .zip fetch, and go.sum verification succeeds through the tunneled checksum db. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014xuhBeTVpfU4SdqVaarvuK * fix(go): address review findings on experimental Go support - Drop fmt/clean from NonDownloadCommands: both load packages via go list and can download modules on a cold cache, which would bypass the proxy under install_only. - Support GOPROXY entries with a base path (e.g. corp Athens/JFrog at https://corp/goproxy): the interceptor now receives host -> base URL and strips the path prefix before parsing module URLs, so verdicts and cooldown key on the real module path. - Default unschemed GOPROXY entries to https, matching go's own behavior, so corp mirrors configured as bare hosts are intercepted instead of silently unanalyzed. - Memoize the final verdict per module zip: go re-requests a failed zip during go get's load phase, which double-recorded stats (the report showed the same blocked module twice) and would have re-prompted on Confirm verdicts. - Fetch .info out-of-band on a cooldown cache miss: go serves .info from its local module cache on any machine that used go before PMG, which silently disabled cooldown. Failure of the side-fetch still fails open for cooldown only. - Move the noop package resolver into packagemanager. Verified live: cold-cache cooldown block now records once; warm-cache rerun is blocked via the side-fetch instead of failing open. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014xuhBeTVpfU4SdqVaarvuK * docs: collapse Go proxy-mode details by default Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014xuhBeTVpfU4SdqVaarvuK --------- Co-authored-by: Claude <noreply@anthropic.com>
95 lines
2.9 KiB
Go
95 lines
2.9 KiB
Go
package packagemanager
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
func TestNormalizeGoProxy(t *testing.T) {
|
|
cases := []struct {
|
|
name string
|
|
goproxy string
|
|
wantChild string
|
|
wantHosts map[string]string
|
|
}{
|
|
{
|
|
name: "empty defaults to public proxy without direct",
|
|
goproxy: "",
|
|
wantChild: "https://proxy.golang.org",
|
|
wantHosts: map[string]string{"proxy.golang.org": "https://proxy.golang.org"},
|
|
},
|
|
{
|
|
name: "default value drops direct",
|
|
goproxy: "https://proxy.golang.org,direct",
|
|
wantChild: "https://proxy.golang.org",
|
|
wantHosts: map[string]string{"proxy.golang.org": "https://proxy.golang.org"},
|
|
},
|
|
{
|
|
name: "pipe separator collapses to comma",
|
|
goproxy: "https://corp.example.com|https://proxy.golang.org|direct",
|
|
wantChild: "https://corp.example.com,https://proxy.golang.org",
|
|
wantHosts: map[string]string{
|
|
"corp.example.com": "https://corp.example.com",
|
|
"proxy.golang.org": "https://proxy.golang.org",
|
|
},
|
|
},
|
|
{
|
|
name: "direct only injects public proxy",
|
|
goproxy: "direct",
|
|
wantChild: "https://proxy.golang.org",
|
|
wantHosts: map[string]string{"proxy.golang.org": "https://proxy.golang.org"},
|
|
},
|
|
{
|
|
name: "off is kept and nothing is intercepted",
|
|
goproxy: "off",
|
|
wantChild: "off",
|
|
wantHosts: map[string]string{},
|
|
},
|
|
{
|
|
name: "proxy with base path keeps full base URL",
|
|
goproxy: "https://corp.example.com:8443/goproxy,direct",
|
|
wantChild: "https://corp.example.com:8443/goproxy",
|
|
wantHosts: map[string]string{"corp.example.com": "https://corp.example.com:8443/goproxy"},
|
|
},
|
|
{
|
|
name: "unschemed entry defaults to https (go behavior)",
|
|
goproxy: "proxy.corp.internal,direct",
|
|
wantChild: "https://proxy.corp.internal",
|
|
wantHosts: map[string]string{"proxy.corp.internal": "https://proxy.corp.internal"},
|
|
},
|
|
{
|
|
name: "unschemed entry with port",
|
|
goproxy: "proxy.corp.internal:8443",
|
|
wantChild: "https://proxy.corp.internal:8443",
|
|
wantHosts: map[string]string{"proxy.corp.internal": "https://proxy.corp.internal:8443"},
|
|
},
|
|
{
|
|
name: "http proxy is kept and intercepted",
|
|
goproxy: "http://insecure.example.com",
|
|
wantChild: "http://insecure.example.com",
|
|
wantHosts: map[string]string{"insecure.example.com": "http://insecure.example.com"},
|
|
},
|
|
{
|
|
name: "file proxy kept verbatim with no host",
|
|
goproxy: "file:///var/goproxy,direct",
|
|
wantChild: "file:///var/goproxy",
|
|
wantHosts: map[string]string{},
|
|
},
|
|
{
|
|
name: "whitespace and empty entries are ignored",
|
|
goproxy: " https://proxy.golang.org , ,direct ",
|
|
wantChild: "https://proxy.golang.org",
|
|
wantHosts: map[string]string{"proxy.golang.org": "https://proxy.golang.org"},
|
|
},
|
|
}
|
|
|
|
for _, tc := range cases {
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
child, hosts := normalizeGoProxy(tc.goproxy)
|
|
assert.Equal(t, tc.wantChild, child)
|
|
assert.Equal(t, tc.wantHosts, hosts)
|
|
})
|
|
}
|
|
}
|