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

[10.x] Add config:show command #47858

Merged
merged 6 commits into from
Jul 28, 2023

Conversation

xiCO2k
Copy link
Contributor

@xiCO2k xiCO2k commented Jul 27, 2023

This PR adds the capability to see the config values directly from your cli.

Output

image

It can also work with nested config values like this:

php artisan config:view auth.guards

Output

image

or, even single values

php artisan config:view auth.guards.web.driver
image

Cheers.

@xiCO2k xiCO2k changed the title [10.x] Add config:show {config} command. [10.x] Add config:show command. Jul 27, 2023
@driesvints driesvints changed the title [10.x] Add config:show command. [10.x] Add config:show command Jul 27, 2023
@driesvints
Copy link
Member

This is neat! But I'm a bit afraid what happens with things like env keys. Outputting these which might get logged seems a little dangerous to me.

@xiCO2k
Copy link
Contributor Author

xiCO2k commented Jul 27, 2023

humm, interesting.

But I think the same happens if you do like cat .env or cat config/auth.php.

@driesvints
Copy link
Member

@xiCO2k good point!

@Bazzly
Copy link

Bazzly commented Jul 27, 2023

This is nice, Hopefully, it will get accepted,

@taylorotwell taylorotwell merged commit 4fbf481 into laravel:10.x Jul 28, 2023
20 checks passed
@taylorotwell
Copy link
Member

Thanks!

@xiCO2k xiCO2k deleted the feat/add-config-show-command branch July 28, 2023 18:12
@andrey-helldar
Copy link
Contributor

https://laravel-idea.com ¯_(ツ)_/¯

--

image

image

@huangdijia
Copy link
Contributor

What effect does the closure display?

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.

None yet

6 participants