Skip to content

Fix: Prevent accidental exposure of local deployment environment files - #786

Closed
mozluk wants to merge 1 commit into
base:mainfrom
mozluk:mozluk-patch-1
Closed

mozluk wants to merge 1 commit into
base:mainfrom
mozluk:mozluk-patch-1

Conversation

@mozluk

@mozluk mozluk commented Aug 21, 2026

Copy link
Copy Markdown

Description

This PR addresses future secret-file exposure risks within the contract-deployments repository.

Vulnerabilities & Security Defects Remediated:

  • Future Secret-File Exposure (.gitignore): Local deployment environment files were not consistently ignored, creating a risk of accidentally committing sensitive configuration or deployment secrets. The repository's .gitignore policy has been updated to explicitly ignore .env, .env.local, and .env.* files. This ensures that any new local environment copies are safely untracked by default.

### Description
This PR addresses future secret-file exposure risks within the `contract-deployments` repository.

**Vulnerabilities & Security Defects Remediated:**
* **Future Secret-File Exposure (`.gitignore`):** Local deployment environment files were not consistently ignored, creating a risk of accidentally committing sensitive configuration or deployment secrets. The repository's `.gitignore` policy has been updated to explicitly ignore `.env`, `.env.local`, and `.env.*` files. This ensures that any new local environment copies are safely untracked by default.
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

@mozluk

mozluk commented Aug 26, 2026

Copy link
Copy Markdown
Author

Hi team, this PR prevents the accidental exposure of local deployment environment files to improve security. Ready for review when you have a moment. Thanks!

@jackchuma jackchuma closed this Sep 22, 2026
@mozluk

mozluk commented Sep 22, 2026

Copy link
Copy Markdown
Author

Hi @jackchuma,

Thanks for taking a look. Just wanted to check if there is an existing internal convention for handling local environment files or if external PRs for repository tooling are not accepted here.

Happy to align with your guidelines!

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.

3 participants