Repository navigation
Document default log output and web server setup - #543
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The file-target migration needs to explain how to preserve log levels that its defaults silently exclude.
1 open finding
What changed in this PR
Clarifies where Yii3 application logs go and how to capture them in different deployments.
Changes:
- Documents default stdout logging and Docker log collection.
- Explains PHP-FPM worker output capture across server recipes.
- Adds instructions for switching to rotating file logs.
| File | Description |
|---|---|
| src/guide/runtime/logging.md | Documents defaults and file-target configuration. |
| src/cookbook/configuring-webservers/nginx.md | Adds PHP-FPM logging guidance. |
| src/cookbook/configuring-webservers/nginx-unit.md | Explains application output capture. |
| src/cookbook/configuring-webservers/lighttpd.md | Adds logging options for PHP deployments. |
| src/cookbook/configuring-webservers/iis.md | Distinguishes application logs from access logs. |
| src/cookbook/configuring-webservers/general.md | Explains output destinations and PHP-FPM capture. |
| src/cookbook/configuring-webservers/apache.md | Links logging guidance for different PHP setups. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Yii3 application templates use
StreamTarget, whose default destination isphp://stdout. They do not create application log files;stderrrequires explicit configuration. This can surprise users deploying without containers, especially because PHP-FPM discards worker output by default.Document the default in the logging guide and keep shared deployment guidance in the general web server guide. Link the five server recipes to that guidance and retain the Nginx-specific caveat. Explain Docker log collection and PHP-FPM output capture, and show how to switch to a rotating file target while retaining all eight log levels.
Disable the Vale acronym definition, heading capitalization, and heading acronym rules.
The default comes from the application logger configuration and the StreamTarget constructor.