-
Notifications
You must be signed in to change notification settings - Fork 367
Description
First off, thank you for creating the go-ldap
library and sharing it with the community.
Using the optional debug logger was helpful while integrating the library. When transitioning from dev/test to a prod version of an application, the debug logger can be disabled, which is super convenient. If a project that imports the library uses a custom logger instance, the library has a fallback to the default log.Logger
instance in certain scenarios, which log the messages to stderr and may not be ideal. It would be helpful to inject a log.Logger
instance into the library to handle these cases in a way that is consistent with the rest of the application.
If this is functionality that you think that others in the community would benefit from, I can submit a PR.