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

[1.x] Made Roadrunner log level configurable #601

Merged
merged 2 commits into from
Oct 24, 2022

Conversation

ejulen
Copy link
Contributor

@ejulen ejulen commented Oct 24, 2022

See #597.

This PR adds a new flag to octane:roadrunner; --log-level, which defaults to the previous behaviour of basing log level on what environment Laravel is currently running in, but now allows for a manual override.

Example: php artisan octane:roadrunner --log-level=info

@driesvints driesvints changed the title Made Roadrunner log level configurable [1.x] Made Roadrunner log level configurable Oct 24, 2022
@taylorotwell taylorotwell merged commit 8380c13 into laravel:1.x Oct 24, 2022
@driesvints
Copy link
Member

@ejulen did you test this? This seems to have broken apps using roadrunner: #602

I'm going to revert this.

driesvints added a commit that referenced this pull request Oct 26, 2022
driesvints added a commit that referenced this pull request Oct 26, 2022
@ejulen
Copy link
Contributor Author

ejulen commented Oct 26, 2022

@driesvints yes, I made a new Octane app locally and tried running with and without the flag. The commit added by @taylorotwell seems to have subtly broken this (seemingly due to operator precedence of concatenation and ?:), just confirmed this locally.

@driesvints
Copy link
Member

Thanks @ejulen. Can you attempt a new PR and explain the breakage from the previous one?

@ejulen
Copy link
Contributor Author

ejulen commented Oct 26, 2022

@driesvints will do!

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