Skip to content

Support user-defined logger instance #365

@n3integration

Description

@n3integration

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions