Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logging redirect is missing for 2.X #2221

Closed
timritzer opened this issue Oct 23, 2018 · 4 comments

Comments

@timritzer
Copy link

commented Oct 23, 2018

The 1.X branch of code offered a method on CBLManager (CBLManager.redirectLogging) That allowed redirecting the logging to be used by the developer for problems in the field (without access to the device).
We built infrastructure that allowed that logging to be uploaded to troubleshoot issues.

CBLog.m features a logging callback, and the underlying couchbase-lite-core supports it well, however the set callback method isn't exposed anymore. (c4log_writeToCallback specifically)

Overwriting the default log callback is an important use case for troubleshooting field devices, but I no longer see a way to call it without a custom build (Something I hesitate to do because it makes EE support less straightforward.


  • Version: Couchbase Lite Swift 2.1
  • Client OS: iOS 12
  • Server: N/A
@pasin

This comment has been minimized.

Copy link
Contributor

commented Oct 23, 2018

Thanks for the feedback. CC: @rajagp

@rajagp

This comment has been minimized.

Copy link
Contributor

commented Oct 24, 2018

@timritzer This is under consideration for our next major release (2.5).

@djpongh djpongh added this to the Iridium milestone Oct 24, 2018

@snej

This comment has been minimized.

Copy link
Member

commented Oct 24, 2018

We are also working on an API that lets CBL generate a log file itself, but exposes the path of the file so the app can upload it. The advantage of that is that the log file is written in a very efficient/compact binary format (which can be decoded by the cblite tool), and CBL will take care of rotating out old log files.

@pasin

This comment has been minimized.

Copy link
Contributor

commented Nov 30, 2018

Let's track this in #1955.

@pasin pasin closed this Nov 30, 2018

@pasin pasin removed the backlog label Nov 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.