Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove the deprecated LogRequest extensions #405

Closed
stijnmoreels opened this issue Jun 13, 2022 · 0 comments · Fixed by #557
Closed

Remove the deprecated LogRequest extensions #405

stijnmoreels opened this issue Jun 13, 2022 · 0 comments · Fixed by #557
Assignees
Labels
breaking-change All issues that require to do a breaking chnage good first issue Good for newcomers requests All issues related to requests
Milestone

Comments

@stijnmoreels
Copy link
Member

stijnmoreels commented Jun 13, 2022

Is your feature request related to a problem? Please describe.
Previously, we were using the UTC Now as start time when tracking requests. We changed this with measuring the request processing with the DurationMeasurement type.

Describe the solution you'd like
Remove the deprecated LogRequest extensions in the Arcus.Observability.Telemetry.Core and Arcus.Observability.Telemetry.AspNetCore projects that still only uses the duration TimeSpan in their signature.

@stijnmoreels stijnmoreels added good first issue Good for newcomers requests All issues related to requests breaking-change All issues that require to do a breaking chnage labels Jun 13, 2022
@stijnmoreels stijnmoreels added this to the v3.0.0 milestone Jun 13, 2022
@stijnmoreels stijnmoreels modified the milestones: v3.0.0, v4.0.0 Sep 15, 2023
@stijnmoreels stijnmoreels self-assigned this Oct 10, 2023
@github-project-automation github-project-automation bot moved this to Done in Roadmap Aug 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
breaking-change All issues that require to do a breaking chnage good first issue Good for newcomers requests All issues related to requests
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant