Skip to content

fix(hooks): use strings for binary file scanning in pre-push#265

Merged
John-David Dalton (jdalton) merged 1 commit intomainfrom
fix/hooks-strings-binary-scanning
Apr 14, 2026
Merged

fix(hooks): use strings for binary file scanning in pre-push#265
John-David Dalton (jdalton) merged 1 commit intomainfrom
fix/hooks-strings-binary-scanning

Conversation

@jdalton
Copy link
Copy Markdown
Collaborator

Summary

  • Use strings command for binary files (WASM, .lockb, etc) instead of raw grep to properly detect embedded paths and secrets
  • Detect binary vs text using grep -qI, then extract printable strings before running pattern checks

Test plan

  • Verify pre-push hook passes on clean push
  • Verify hook catches hardcoded paths in binary files

@jdalton John-David Dalton (jdalton) force-pushed the fix/hooks-strings-binary-scanning branch from d5800d9 to 67af68a Compare April 14, 2026 00:47
@jdalton John-David Dalton (jdalton) merged commit 7ff59f2 into main Apr 14, 2026
13 checks passed
@jdalton John-David Dalton (jdalton) deleted the fix/hooks-strings-binary-scanning branch April 14, 2026 00:59
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