Skip to content

Migrate DebugLogConfigurator to Use New Logger Infrastructure and Isolate from Timber, While Supporting Timber Setup in Android Module#9863

Merged
wmontwe merged 2 commits into
thunderbird:mainfrom
shamim-emon:fix-issue-9573
Oct 2, 2025
Merged

Migrate DebugLogConfigurator to Use New Logger Infrastructure and Isolate from Timber, While Supporting Timber Setup in Android Module#9863
wmontwe merged 2 commits into
thunderbird:mainfrom
shamim-emon:fix-issue-9573

Conversation

@shamim-emon

@shamim-emon shamim-emon commented Sep 27, 2025

Copy link
Copy Markdown
Collaborator

Implementation Highlights:

  • This was originally addressed in 9816, but that change introduced a side effect: since the Timber. DebugTree were no longer being planted, logs were no longer collected.
  • This PR migrates DebugLogConfigurator to the new Logger infrastructure and adds an Android-specific platform configuration, invoked from the common code, to ensure Timber is properly set up.

@shamim-emon shamim-emon force-pushed the fix-issue-9573 branch 7 times, most recently from 61fc1d4 to 119e639 Compare October 2, 2025 02:22
@shamim-emon shamim-emon requested a review from wmontwe October 2, 2025 03:37

@wmontwe wmontwe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a tiny rename left.

remove the DebugLogConfigurator from the exclusion list of Timber logging, present in the ValidateLogger#no class should use Timber logging quality test.

include the PlatformInitializer.android in the exclusion list of Timber logging, present in the ValidateLogger#no class should use Timber logging quality test.

@wmontwe wmontwe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wmontwe wmontwe merged commit 3b78eb2 into thunderbird:main Oct 2, 2025
12 of 13 checks passed
@thunderbird-botmobile thunderbird-botmobile Bot added this to the Thunderbird 14 milestone Oct 2, 2025
@shamim-emon shamim-emon deleted the fix-issue-9573 branch October 2, 2025 14:31
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.

Move DebugLogConfigurator into new logger infrastructure

2 participants