Skip to content

fix: local env files should not be baked in - #2230

Merged
bosbaber merged 2 commits into
mainfrom
stephan/int1-686
Aug 13, 2026
Merged

fix: local env files should not be baked in#2230
bosbaber merged 2 commits into
mainfrom
stephan/int1-686

Conversation

@bosbaber

@bosbaber bosbaber commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

We noticed that the local environment .env files gets baked into builds causing misconfigurations in production builds.

This PR cleans this up.

@bosbaber bosbaber self-assigned this Aug 13, 2026
@bosbaber
bosbaber requested review from dragosp1011 and a lite review from Copilot August 13, 2026 08:11
@bosbaber
bosbaber marked this pull request as ready for review August 13, 2026 08:11
dragosp1011
dragosp1011 previously approved these changes Aug 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to prevent local .env configuration files from being included in Docker build contexts, which can lead to production images being built with unintended local settings (notably when frontend builds load .env.local during compilation).

Changes:

  • Updates .dockerignore to exclude the wallet frontend’s .env.local from Docker build contexts.
  • Adds inline documentation explaining why .env.local must not be included in image builds.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .dockerignore Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@bosbaber
bosbaber merged commit 9a59f56 into main Aug 13, 2026
14 checks passed
@bosbaber
bosbaber deleted the stephan/int1-686 branch August 13, 2026 08:20
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants