Skip to content

feat: implement security fixes for critical vulnerabilities#26

Merged
zhujian0805 merged 2 commits intoChat2AnyLLM:mainfrom
zhujian0805:main
Dec 25, 2025
Merged

feat: implement security fixes for critical vulnerabilities#26
zhujian0805 merged 2 commits intoChat2AnyLLM:mainfrom
zhujian0805:main

Conversation

@zhujian0805
Copy link
Copy Markdown
Collaborator

  • add sec report
  • feat: implement security fixes for critical vulnerabilities

James Zhu and others added 2 commits December 25, 2025 04:58
- Fix command injection vulnerability by replacing shell=True with shell=False in goose.py
- Add path traversal protection using os.path.realpath() and os.path.commonpath() in agents/base.py
- Secure configuration file permissions using file opener with 600 permissions in goose.py
- Implement SecureAPIKeyHandler for safer API key handling in env_builder.py
- Add path validation for config files with _validate_safe_path() function in config.py
- Prevent insecure deserialization by validating paths before file access

These changes address all critical vulnerabilities identified in the security audit report.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@zhujian0805 zhujian0805 merged commit 9caf1ee into Chat2AnyLLM:main Dec 25, 2025
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