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

Allow external loggers to be used with glog as default #148

Merged
merged 1 commit into from
Nov 2, 2024

Conversation

janhoon
Copy link
Contributor

@janhoon janhoon commented Oct 26, 2024

This is a proposal for #75

Any logger can now be added by setting ginoauth2.DefaultLogger = MyCustomLogger as long as the logger implements the ginoauth2.Logger interface.

@szuecs szuecs added the architectural all changes in the hot path, big changes, control flow changes label Oct 29, 2024
@szuecs
Copy link
Member

szuecs commented Oct 29, 2024

Can you please sign-off your commits and squash into one?
Thanks 🙏🏽

@janhoon
Copy link
Contributor Author

janhoon commented Oct 29, 2024

will do 🫡

@janhoon janhoon force-pushed the master branch 2 times, most recently from d8550ed to 823b46b Compare October 29, 2024 22:31
@janhoon
Copy link
Contributor Author

janhoon commented Oct 29, 2024

Signed and squashed 🫡

@szuecs
Copy link
Member

szuecs commented Oct 30, 2024

👍

1 similar comment
@RomanZavodskikh
Copy link
Member

👍

@szuecs szuecs merged commit 66a94d4 into zalando:master Nov 2, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architectural all changes in the hot path, big changes, control flow changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants