Skip to content

Conversation

continue[bot]
Copy link
Contributor

@continue continue bot commented Sep 17, 2025

Config files containing API keys and other sensitive data should not be world-readable. This PR changes the file creation mode to 0600 (owner read/write only) when creating or updating config.yaml files.

Changes:

  • Modified getConfigYamlPath() in core/util/paths.ts to set mode 0600 when creating initial config.yaml
  • Modified editConfigYaml() in core/util/paths.ts to set mode 0600 when updating config.yaml
  • Modified config conversion command in extensions/vscode/src/commands.ts to set mode 0600

Security Impact:
This prevents accidentally exposing API keys and other credentials in config.yaml to other users on multi-user systems.


Summary by cubic

Set config.yaml permissions to 0600 (owner read/write only) when creating or updating the file in both core and VS Code extension paths. This prevents world-readable configs and reduces the risk of leaking API keys on multi-user systems.

Config files containing API keys and other sensitive data should not be
world-readable. Changed fs.writeFileSync calls to use mode 0o600 (owner
read/write only) when creating or updating config.yaml files.

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <noreply@continue.dev>
@continue continue bot requested a review from a team as a code owner September 17, 2025 22:13
@continue continue bot requested review from tingwai and removed request for a team September 17, 2025 22:13
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 17, 2025
Copy link

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant