Skip to content

Configurable log options#5570

Closed
miloslavkostir wants to merge 1 commit intoBookStackApp:developmentfrom
miloslavkostir:development
Closed

Configurable log options#5570
miloslavkostir wants to merge 1 commit intoBookStackApp:developmentfrom
miloslavkostir:development

Conversation

@miloslavkostir
Copy link

In my Laravel projects I'm used to create log options configurable. Projects run in Docker container and the log dir is mapped out of the container.

@ssddanbrown
Copy link
Member

Thanks for offering this PR @miloslavkostir, but I'm not keen on adding extra env options without proven wider need.

Not sure why you can't map the existing directory/file out of a container already.
Otherwise, some other options are:

  • Patch the option as part of your container build process.
  • Use the logical theme system to override the option.

@ssddanbrown ssddanbrown closed this Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants