Skip to content

CBL-9167 : Leave the TLS log domain at the LiteCore default level - #523

Merged
pasin merged 1 commit into
release/4.0from
fix/CBL-9167
Oct 21, 2026
Merged

pasin merged 1 commit into
release/4.0from
fix/CBL-9167

Conversation

@pasin

@pasin pasin commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator
  • Stop setting the TLS log level, so mbedTLS logging stays at the
    LiteCore default (Warning) instead of following the app's level.
  • Keep the TLS domain mapped to NETWORK.
  • Add MAPPED_ONLY_C4_LOGGING_DOMAINS for LiteCore domains that are
    mapped but whose level is not set.
  • This is in line with the .NET and iOS behavior.

@pasin
pasin requested a review from a team October 21, 2026 02:48
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

This is a release branch and commits are restricted.

Please confirm this PR is one of the following:

  • A response to a customer ask
  • An approved backport
  • A non-functional change (e.g. changes needed for building an older version)
  • A bug fix for a x.y.0 release
  • A change that has been granted an exception (please comment)

@pasin pasin assigned borrrden and unassigned borrrden Oct 21, 2026
- Stop setting the TLS log level, so mbedTLS logging stays at the
  LiteCore default (Warning) instead of following the app's level.
- Keep the TLS domain mapped to NETWORK.
- Add MAPPED_ONLY_C4_LOGGING_DOMAINS for LiteCore domains that are
  mapped but whose level is not set.
- This is in line with the .NET and iOS behavior.
@borrrden borrrden assigned pasin and unassigned borrrden Oct 21, 2026
@pasin
pasin merged commit 216a87d into release/4.0 Oct 21, 2026
@pasin
pasin deleted the fix/CBL-9167 branch October 21, 2026 15:07
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.

3 participants