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

Remove accepted_comparators option from logger interceptors #326

Merged
merged 9 commits into from
Jul 10, 2023

Conversation

guisehn
Copy link
Contributor

@guisehn guisehn commented Jul 5, 2023

Remove accepted_comparators option from logger interceptors.

Fixes #325

Restores the default behavior before #227 was merged

@guisehn guisehn requested a review from polvalente July 6, 2023 18:08
@guisehn guisehn changed the title Reverse logger level comparison logic Remove accepted_comparators in logger interceptors Jul 7, 2023
@guisehn guisehn changed the title Remove accepted_comparators in logger interceptors Remove accepted_comparators option from logger interceptors Jul 7, 2023
@polvalente polvalente merged commit d5ad40a into elixir-grpc:master Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Default accepted_comparators option of logger interceptors doesn't match Elixir's default behavior
2 participants