Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix not detecting custom envfile in /tmp/envfile #595

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JofArnold
Copy link

@JofArnold JofArnold commented Jul 15, 2021

Fixes issue where if an envfile was available in /tmp/envfile it was incorrectly being File.read'd causing the begin block to throw and thus return an empty object to ReadDotEnv.rb

I gather this fixes a decent number of issues for instance #511 #594 and no doubt a few others assuming those people have a pre-build step that writes their env file of choice to /tmp/envfile.

Note: I'm not a ruby coder!

Fixes issue where if an envfile was available in /tmp/envfile it was incorrectly being File.read'd causing the begin block to throw and thus return an empty object to ReadDotEnv.rb
@yurykorzun
Copy link

Can this fix be merged? I keep having this file blocked during the build

@heath-clink
Copy link

/tmp/envfile is supposed to contain a filename of an env file, not the contents of an env file. This change will break existing apps that correctly utilize /tmp/envfile.

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