-
Notifications
You must be signed in to change notification settings - Fork 1
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
Create Logger Abstraction for Ably Chat SDK #20
Comments
Thanks for specifying the Requirements. We can both agree that the logging helps to debug and understand the execution flow of the application. I would like to mention a few points that should help us achieve this goal.
I would leave implementation of |
Btw, in dotnet we also use localized loggers with parent context, |
We need to create a flexible and configurable logger abstraction for the Ably Chat SDK that integrates smoothly with Android’s default logging system and allows for custom logging handlers to be set via Chat Client's
ClientOptions
. This will provide users with better control over logging behavior and allow them to intercept or handle logs in a way that suits their needs.Requirements:
By default, the SDK should use Android’s native logging system (e.g.,
Log.d
,Log.i
, etc.) for logging events.ClientOptions
:logHandler
property in theClientOptions
with the type(message: string, level: LogLevel, context: LogContext?) -> Unit
logLevel
property that filters logs based on log levelChatClient
Instances:ChatClient
are created, each instance should have its own separate logger handler if specified in the respectiveClientOptions
.slf4j
, we are going to make Chat SDK JVM library (once we refactorably-java
)┆Issue is synchronized with this Jira Task by Unito
The text was updated successfully, but these errors were encountered: