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

Update HTTP in and HTTP out logging #4155

Closed
JamesNK opened this issue Jul 7, 2023 · 4 comments
Closed

Update HTTP in and HTTP out logging #4155

JamesNK opened this issue Jul 7, 2023 · 4 comments
Labels
area-telemetry

Comments

@JamesNK
Copy link
Member

JamesNK commented Jul 7, 2023

.NET 8 improves built-in logging for HTTP in and HTTP out. Update packages in dotnet/extensions to use built-in logging and apply redaction and other features to them.

Blocked on:

@Tratcher What is the state of HTTP logging in ASP.NET Core? Are required changes made there? Has dotnet/extensions been updated to use it?

cc @joperezr
@Tratcher
Copy link
Member

Tratcher commented Jul 7, 2023

I have draft PRs but haven't been able to work on them recently. dotnet/aspnetcore#48679 #4056. Not sure if they'll be a priority for this release.

@xakep139
Copy link
Contributor

Outgoing part is covered within #4211

@xakep139
Copy link
Contributor

Incoming part is covered in #4210

@xakep139
Copy link
Contributor

@klauco @geeknoid should we close this issue since we have two specific ones?

@ghost ghost locked as resolved and limited conversation to collaborators Sep 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area-telemetry
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@JamesNK @Tratcher @RussKie @xakep139 @geeknoid