Skip to content

Add .gitignore entries for sensitive files#508

Open
pete-builds wants to merge 2 commits intowmo-raf:mainfrom
pete-builds:fix-gitignore
Open

Add .gitignore entries for sensitive files#508
pete-builds wants to merge 2 commits intowmo-raf:mainfrom
pete-builds:fix-gitignore

Conversation

@pete-builds
Copy link
Copy Markdown

The .gitignore is missing entries for private keys and credentials (*.pem, *.key, *.p12, *.pfx, credentials.json, service-account*.json). Added those to help prevent accidental commits of sensitive files.

Copy link
Copy Markdown

@s6r6k s6r6k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cld I please ask why .env and .env.local is not added?

@pete-builds
Copy link
Copy Markdown
Author

Good catch, you're right. Adding .env and .env.local now.

@pete-builds
Copy link
Copy Markdown
Author

Actually, I take that back. Looking at the full .gitignore, .env is already covered under the Environments section (line ~126), and .env.local and .env.dev are already listed near the bottom. So those are already handled. This PR just adds the key/credential patterns that were missing.

@pete-builds
Copy link
Copy Markdown
Author

Revised — removed the credential and certificate entries that aren't relevant to this project.

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.

2 participants