Replies: 2 comments
-
This is part of an ongoing feature where we will eventually remove the parameterless extension #291 and replace it with a new one with the HTTP-specific options #293. |
Beta Was this translation helpful? Give feedback.
0 replies
-
I'll move this to an discussion, as we already have issues for this. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
We have made the parameterless
AddHttpCorrelation
method obsolete in favor of theAddHttpCorrelation
method which takes aHttpCorrelationInfoOptions
object as parameter.However, I've seen that, when you want to use the default implementation, you now have to call this method like this:
I think this is a bit cumbersome and pollutes the code.
If you want to stick with the defaults, why not just leave it as is and let people call this just like:
This is simple and straightforward.
We can keep the parameterless
AddHttpCorrelation
overload. This parameterless overload can be implemented like this:First mentionned here
Beta Was this translation helpful? Give feedback.
All reactions