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

Cannot use .env encrypt if telescope is enabled #1262

Closed
pyr0hu opened this issue Oct 18, 2022 · 4 comments · Fixed by laravel/framework#44758
Closed

Cannot use .env encrypt if telescope is enabled #1262

pyr0hu opened this issue Oct 18, 2022 · 4 comments · Fixed by laravel/framework#44758
Assignees
Labels

Comments

@pyr0hu
Copy link

pyr0hu commented Oct 18, 2022

  • Telescope Version: 4.9.4
  • Laravel Version: 9.33.0
  • PHP Version: 8.1
  • Database Driver & Version: Does not matter

Description:

I want to use the new environment file encryption feature in Laravel. If I call php artisan env:encrypt --env=staging , it fails because it tries to connect to the staging instance's Redis but from my local computer I cannot access that. The staging instance has Telescope enabled (as it's on an company-wide network and it helps with debugging). But Laravel loads the whole environment from the .env.staging file, therefore overwriting my local Redis configuration. Telescope will try to use the staging's Redis connection to check for pause-recording and all of these stuff. As it cannot connect, the whole encryption command fails with a timeout error.

Steps To Reproduce:

.env.staging:
TELESCOPE_ENABLED=true
REDIS_HOST=unaccessible redis host

php artisan env:encrypt --env=staging

@driesvints
Copy link
Member

Gonna let @joedixon check into this when he's back.

@joedixon
Copy link

I have just submitted a PR which should resolve this. It would great if you could run a quick test with your setup @pyr0hu using the PR to ensure it works for you: laravel/framework#44758

@pyr0hu
Copy link
Author

pyr0hu commented Oct 27, 2022

@joedixon Yup, using that fix works!

@joedixon
Copy link

@pyr0hu thanks for testing so quickly!

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 a pull request may close this issue.

3 participants