Skip to content

codex support, additional security options, add some limits#1

Open
timlevett wants to merge 10 commits intoterraboops:mainfrom
timlevett:main
Open

codex support, additional security options, add some limits#1
timlevett wants to merge 10 commits intoterraboops:mainfrom
timlevett:main

Conversation

@timlevett
Copy link
Copy Markdown

@timlevett timlevett commented Mar 18, 2026

  • Adds in codex support with examples
  • adds in further security options (token access, localhost binding, logging option in hook)
  • Adds CLAUDE.md (let me know if you want to drop this)

timlevett and others added 10 commits March 17, 2026 09:28
- Bind to 127.0.0.1 by default (add --bind flag for network access)
- Add optional --token flag for Bearer auth on all endpoints
- Validate webhook input lengths (event, session, tool, client_key)
- Guard base64/PNG decode with size limits (32KB b64, 16KB raw)
- Cap unique_agents HashSet at MAX_UNIQUE_AGENTS (10,000) to prevent memory exhaustion

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add a "Secure setup" section to the README explaining the --token flag,
Bearer auth header, and how to configure LFG_TOKEN in the shell profile.
Update lfg-hook.sh to read LFG_TOKEN and forward it when set.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Harden server security with auth token, input validation, and limits
The old [[hooks.*]] TOML entries in config.toml are not fired by current
Codex versions. Switch to the correct schema:
- config.toml: only [features] codex_hooks = true
- hooks.json: hook definitions with PascalCase event names (SessionStart,
  PreToolUse, PostToolUse, Stop)

Also update lfg-hook.sh default URL to 127.0.0.1 (confirmed working) and
update README with correct two-file setup instructions.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Update Codex hook config to use hooks.json with PascalCase format
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