Skip to content

Added logger implementation for components logs #42

Merged
pacific-ring merged 3 commits into
mainfrom
task/fix-logs
Jun 29, 2026
Merged

Added logger implementation for components logs #42
pacific-ring merged 3 commits into
mainfrom
task/fix-logs

Conversation

@pacific-ring

Copy link
Copy Markdown
Contributor

Description

Routes all SDK-internal logging through a new static Logger facade (Logger.kt) instead of calling android.util.Log directly

  • Adds object Logger with d/i/v/w/e methods, each gated on a single private enabled flag that is off by default, so a published SDK is silent unless explicitly switched on.
  • The flag is mutable only via an internal setEnabled() — application code cannot turn on token/session logging, keeping the logic inside the SDK boundary. The switch is intentionally not yet wired to any public initialization option -- the seam exists for a future change to decide how diagnostics get enabled.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvement

References

  • support ticket
  • community post
  • StackOverflow post
  • support forum thread

Testing

  • Unit tests
  • Manual testing
  • Sample app testing

Test Configuration:

  • Android version(s):
  • Device(s):

Screenshots (if appropriate)

Checklist

@pacific-ring pacific-ring requested a review from a team as a code owner June 24, 2026 13:00
@pacific-ring pacific-ring changed the title [Do Not Merge] UIC-947 : Added logger implementation Added logger implementation for components logs Jun 29, 2026
@pacific-ring pacific-ring requested a review from pmathew92 June 29, 2026 10:01
@pacific-ring pacific-ring merged commit 102169a into main Jun 29, 2026
5 checks passed
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.

2 participants