Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Safe-load .env only if exists to prevent fatal error during site install. #589

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 3, 2021
Merged

Safe-load .env only if exists to prevent fatal error during site install. #589

merged 1 commit into from
Jan 3, 2021

Conversation

normanlolx
Copy link
Collaborator

Seems #584 accidentally swapped safe-loading (only if exists) the .env file with a simple load() which now causes a fatal on drush site:install. Let's switch load() back to safeLoad().

$ drush -y si --account-name=admin --account-pass=admin

Fatal error: Uncaught Dotenv\Exception\InvalidPathException: Unable to read any of the environment file(s) at [/Users/leymannx/Sites/d9/.env]. in /Users/leymannx/Sites/d9/vendor/vlucas/phpdotenv/src/Store/FileStore.php:68
Stack trace:
#0 /Users/leymannx/Sites/d9/vendor/vlucas/phpdotenv/src/Dotenv.php(222): Dotenv\Store\FileStore->read()
#1 /Users/leymannx/Sites/d9/load.environment.php(18): Dotenv\Dotenv->load()
#2 /Users/leymannx/Sites/d9/vendor/composer/autoload_real.php(75): require('/Users/leymannx...')
#3 /Users/leymannx/Sites/d9/vendor/composer/autoload_real.php(65): composerRequire62f8ceaa89421c6aefe473318c51d739('d511210698f02d8...', '/Users/leymannx...')
#4 /Users/leymannx/Sites/d9/vendor/autoload.php(7): ComposerAutoloaderInit62f8ceaa89421c6aefe473318c51d739::getLoader()
#5 /Users/leymannx/Sites/d9/vendor/drush/drush/drush.php(56): include_once('/Users/leymannx...')
#6 /Users/leymannx/Sites/d9/vendor/drush/drush/includes/preflight.inc(18): require('/Users/leymannx...')
#7 phar:///usr/local/bin/drush/bin/drush.php in /Users/leymannx/Sites/d9/vendor/vlucas/phpdotenv/src/Store/FileStore.php on line 68

@normanlolx normanlolx requested a review from webflo December 27, 2020 15:48
@owenbush
Copy link

Yes, I stumbled across this today too, this is blowing up my CircleCI pipelines for Drupal 9 builds.

Fatal error: Uncaught Dotenv\Exception\InvalidPathException: Unable to read any of the environment file(s) at [/home/circleci/project/build/.env]. in /home/circleci/project/build/vendor/vlucas/phpdotenv/src/Store/FileStore.php on line 68

Dotenv\Exception\InvalidPathException: Unable to read any of the environment file(s) at [/home/circleci/project/build/.env]. in /home/circleci/project/build/vendor/vlucas/phpdotenv/src/Store/FileStore.php on line 68

Call Stack:
    0.0001     398432   1. {main}() /home/circleci/project/build/vendor/drush/drush/drush:0
    0.0004     410216   2. require('/home/circleci/project/build/vendor/drush/drush/drush.php') /home/circleci/project/build/vendor/drush/drush/drush:4
    0.0005     411432   3. include_once('/home/circleci/project/build/vendor/autoload.php') /home/circleci/project/build/vendor/drush/drush/drush.php:56
    0.0007     425736   4. ComposerAutoloaderInit79ff7fd1b7eb8ce20350371ca2fbb080::getLoader() /home/circleci/project/build/vendor/autoload.php:7
    0.0133    1875200   5. composerRequire79ff7fd1b7eb8ce20350371ca2fbb080($fileIdentifier = 'd511210698f02d87ca48e3972f64323e', $file = '/home/circleci/project/build/vendor/composer/../../load.environment.php') /home/circleci/project/build/vendor/composer/autoload_real.php:65
    0.0134    1876624   6. require('/home/circleci/project/build/load.environment.php') /home/circleci/project/build/vendor/composer/autoload_real.php:75
    0.0173    2148368   7. Dotenv\Dotenv->load() /home/circleci/project/build/load.environment.php:18
    0.0173    2148368   8. Dotenv\Store\FileStore->read() /home/circleci/project/build/vendor/vlucas/phpdotenv/src/Dotenv.php:222

@normanlolx normanlolx merged commit 0e88ba1 into 9.x Jan 3, 2021
@normanlolx normanlolx deleted the dotenv_safeload branch January 3, 2021 10:29
@normanlolx normanlolx mentioned this pull request Jan 5, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@webflo webflo Awaiting requested review from webflo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@normanlolx @owenbush