Skip to content

feat: warn when the config is not ignored by git - #7

Closed
yepzdk wants to merge 1 commit into
mainfrom
feature/warn-git-exposed-config
Closed

yepzdk wants to merge 1 commit into
mainfrom
feature/warn-git-exposed-config

Conversation

@yepzdk

@yepzdk yepzdk commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator

Agent Proxy writes API keys in plain text. With the config in a git repository that does not ignore it, one git commit -a publishes them; itkdev-docker's tracked traefik/dynamic-conf.yaml was such a case. A warning now shows above the list while that holds.

The check looks for .git before running git, because /usr/bin/git on a Mac without the developer tools opens an install prompt; without git there is no warning. In directory mode it checks a would-be service.yml, so a .gitignore that lists service files one by one still warns.

@yepzdk

yepzdk commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator Author

Combined into #9.

@yepzdk yepzdk closed this Oct 15, 2026
@yepzdk
yepzdk deleted the feature/warn-git-exposed-config branch October 15, 2026 05:41
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