Skip to content

Set default log levels for fresh and existing installs#10483

Open
Montekkundan wants to merge 1 commit into
aces:mainfrom
Montekkundan:montek/log-level-defaults
Open

Set default log levels for fresh and existing installs#10483
Montekkundan wants to merge 1 commit into
aces:mainfrom
Montekkundan:montek/log-level-defaults

Conversation

@Montekkundan
Copy link
Copy Markdown

  • Seed default log-level config values for fresh installs:
    • database_log_level: warning
    • request_log_level: warning
    • exception_log_level: warning
    • profiler_log_level: none
  • Add a SQL patch to backfill missing, NULL, or blank log-level values on existing installs without overriding explicit values.
  • Update runtime log-level fallback behavior so missing values use predictable defaults instead of silently falling back to none.
  • Use NullLogger when request logging is explicitly set to none.

Testing instructions

  • vendor/bin/phpunit --configuration test/phpunit.xml test/unittests/LogSettingsTest.php

#10401

@github-actions github-actions Bot added Language: SQL PR or issue that update SQL code Language: PHP PR or issue that update PHP code Module: electrophysiology_browser PR or issue related to electrophysiology_browser module labels May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: PHP PR or issue that update PHP code Language: SQL PR or issue that update SQL code Module: electrophysiology_browser PR or issue related to electrophysiology_browser module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate NULL log-level values after install and define defaults for debugging

2 participants