docs(logging): correct LB_LOGGING_* names#991
Conversation
Was `LB_LOG_*` but should be `LB_LOGGING_*`
There was a problem hiding this comment.
Pull request overview
Updates Docker environment variable documentation for logging settings to use the correct LB_LOGGING_* names (instead of LB_LOG_*) across the docs.
Changes:
- Renamed documented logging env vars to
LB_LOGGING_FOLDER,LB_LOGGING_LEVEL, andLB_LOGGING_SQLin installation docs. - Renamed documented logging env vars to
LB_LOGGING_FOLDER,LB_LOGGING_LEVEL, andLB_LOGGING_SQLin basic configuration docs.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/source/INSTALLATION.rst | Corrects documented logging env var names for Docker configuration. |
| docs/source/BASIC-CONFIGURATION.rst | Corrects documented logging env var names for Docker configuration. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| ``LB_LOG_FOLDER`` | ||
| ``LB_LOGGING_FOLDER`` | ||
| Log directory (default: ``/var/log/librebooking``) |
There was a problem hiding this comment.
The documented default log directory here (/var/log/librebooking) doesn’t match the actual default used by the app and the provided .env.example (/var/log/librebooking/log). Updating this default in the docs would prevent misconfiguration/confusion for Docker users.
| Log directory (default: ``/var/log/librebooking``) | |
| Log directory (default: ``/var/log/librebooking/log``) |
|
|
||
| ``LB_LOG_FOLDER`` | ||
| ``LB_LOGGING_FOLDER`` | ||
| Log directory (default: ``/var/log/librebooking``) |
There was a problem hiding this comment.
The documented default log directory here (/var/log/librebooking) doesn’t match the actual default used by the app and the provided .env.example (/var/log/librebooking/log). Updating this default in the docs would prevent misconfiguration/confusion for Docker users.
| Log directory (default: ``/var/log/librebooking``) | |
| Log directory (default: ``/var/log/librebooking/log``) |
Was
LB_LOG_*but should beLB_LOGGING_*