Skip to content

Commit 6216e6d

Browse files
committed
fix(security): remove GIT_SSL_NO_VERIFY=true from sfw-free shims
The sfw-free shims were injecting `export GIT_SSL_NO_VERIFY=true` into every generated bash shim, disabling SSL certificate verification for all git operations performed by wrapped package manager commands (npm, yarn, pnpm, pip, uv, cargo). This exposed users to man-in-the-middle attacks on any git-over-HTTPS connection. Remove the workaround entirely — disabling TLS verification is far more dangerous than the GIT_SSL_CAINFO issue it was intended to work around.
1 parent e9a48c5 commit 6216e6d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

.claude/hooks/setup-security-tools/index.mts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -273,10 +273,6 @@ async function setupSfw(apiKey: string | undefined): Promise<boolean> {
273273
'fi',
274274
)
275275
}
276-
if (!isEnterprise) {
277-
// Workaround: sfw-free does not yet set GIT_SSL_CAINFO (temporary).
278-
bashLines.push('export GIT_SSL_NO_VERIFY=true')
279-
}
280276
bashLines.push(`exec "${binaryPath}" "${realBin}" "$@"`)
281277
const bashContent = bashLines.join('\n') + '\n'
282278
const bashPath = path.join(shimDir, cmd)

0 commit comments

Comments
 (0)