Skip to content

7.x: Fix console on bare-metal installs#2037

Open
tvdijen wants to merge 1 commit into
OpenConext:release/7.1from
tvdijen:patch-1
Open

7.x: Fix console on bare-metal installs#2037
tvdijen wants to merge 1 commit into
OpenConext:release/7.1from
tvdijen:patch-1

Conversation

@tvdijen

@tvdijen tvdijen commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Open for discussion, because I don't know if this is the right solution, but we ran into the issue that the .env file was not loaded automatically when running bin/console commands. We run multiple instances of EB on the same machine, so it's crucial that we can set the environment variables (especially APP_SECRET) per instance instead of setting it globally. This change seems to fix that.

Open for discussion, because I don't know if this is the right solution, but we ran into the issue that the .env file was not loaded automatically when running `bin/console` commands.
We run multiple instances of EB on the same machine, so it's crucial that we can set the environment variables (especially APP_SECRET) per instance instead of setting it globally.  This change seems to fix that.
@tvdijen tvdijen changed the title Fix console on bare-metal installs 7.x: Fix console on bare-metal installs Jun 30, 2026
@baszoetekouw baszoetekouw requested a review from johanib July 2, 2026 07:11

@johanib johanib 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.

Thanks. I took a look, this seems a fine fix 👍 It's narrowly scoped, so should not break anything.

As a sidenote, I noticed a bug in the dotenv not loading .env override files, for example .env.prod is ignored. Made a separate pr for that: #2042

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