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
Adjust public contract of Logger Middlewares #6927
base: main
Are you sure you want to change the base?
Conversation
@http4s/core hey team! I apologize for bothering you, I just want a little attention paid to this PR. It's not rocket science, but I really feel that exposing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable to me. Maybe it's better called logWithEntity
instead of logWithBody
?
Good call, will change it, thanks! |
Sorry for pushing this so hard, but can we do another review iteration? There's anxiety, this can end up with a ton of conflicts due to the forthcoming #7122... |
Yeah, sorry for kvetching. It's the correct ordering. I merely tried to overtake you! 🙈 |
Stream[F, Byte]
is an implementation detail of a specific variant ofEntity[F]
, so let's adjust the public contract of Logger Middlewares. Naming is the second hard thing in programming, so I'm fully open to debate on it.