Skip to content

fix: remediate CodeQL security alerts#361

Merged
santoshkumarradha merged 1 commit intomainfrom
codex/codeql-alert-remediation
Apr 7, 2026
Merged

fix: remediate CodeQL security alerts#361
santoshkumarradha merged 1 commit intomainfrom
codex/codeql-alert-remediation

Conversation

@santoshkumarradha
Copy link
Copy Markdown
Member

Summary

  • remove registration-time callback health probing and replace it with syntax-only normalization/selection to eliminate uncontrolled outbound requests from node registration
  • gate serverless /discover fetches behind a config-backed host allowlist with safe local defaults, redirect blocking, and YAML/env support
  • replace direct SHA-256 passphrase hashing with a versioned PBKDF2-based encryption format and add focused regression tests
  • tighten the flagged Python and functional test cases and add explicit least-privilege workflow permissions

CodeQL alerts addressed

Validation

  • cd control-plane && go test ./internal/encryption ./internal/handlers ./internal/server ./cmd/af ./cmd/agentfield-server
  • cd sdk/python && python3 -m pytest tests/test_agent_networking.py
  • python3 -m py_compile tests/functional/tests/test_ts_agent.py
  • git diff --check

@santoshkumarradha santoshkumarradha requested review from a team and AbirAbbas as code owners April 7, 2026 18:38
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 7, 2026

Performance

SDK Memory Δ Latency Δ Tests Status
Python 7.9 KB -13% 0.43 µs +23%

✓ No regressions detected

@santoshkumarradha santoshkumarradha merged commit f7f5abc into main Apr 7, 2026
34 checks passed
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