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.

Sign up for GitHub

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

Jump to bottom

Remove HttpCorrelation regular correlation options constructor #292

Closed
stijnmoreels opened this issue Feb 3, 2022 · 0 comments · Fixed by #445
Closed

Remove HttpCorrelation regular correlation options constructor #292

stijnmoreels opened this issue Feb 3, 2022 · 0 comments · Fixed by #445
Assignees
@stijnmoreels
Labels
breaking-change All issues related to breaking changes correlation All issues related to HTTP correlation good first issue Good for newcomers
Milestone
v2.0

Comments

@stijnmoreels
Copy link
Member

Is your feature request related to a problem? Please describe.
When designing the HTTP correlation, we chose to use HTTP-specific options, making the HttpCorrelation constructor with the regular CorrelationInfoOptions obsolete.

Describe the solution you'd like
Remove the HttpCorrelation constructor with the regular CorrelationInfoOptions.
@stijnmoreels stijnmoreels added breaking-change All issues related to breaking changes correlation All issues related to HTTP correlation labels Feb 3, 2022
@stijnmoreels stijnmoreels added this to the v2.0 milestone Feb 3, 2022
@stijnmoreels stijnmoreels added the good first issue Good for newcomers label Feb 3, 2022
@vincenttermaat vincenttermaat mentioned this issue Jun 9, 2022
Closed
@stijnmoreels stijnmoreels mentioned this issue Apr 11, 2023
Merged
@stijnmoreels stijnmoreels self-assigned this Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stijnmoreels stijnmoreels

Labels
breaking-change All issues related to breaking changes correlation All issues related to HTTP correlation good first issue Good for newcomers
Projects
None yet
Milestone
v2.0
1 participant
@stijnmoreels