-
Notifications
You must be signed in to change notification settings - Fork 10.3k
Add EventSource logging to RateLimitingMiddleware #41266
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
Comments
What are we adding? Can we list the events and counters here? I wonder if we should switch to metrics in .NET 7? cc @noahfalk |
@wtgodbe Do we have a set of things we want to add for this? Will it happen in preview6? |
Not yet - I'm planning to do it in the endpoint PR, unless that would put that PR at risk of missing preview6. In that case I'd do it in a separate PR for preview7 |
Thanks for contacting us. We're moving this issue to the |
Follow-up to #41008, part of #37384. See https://github.com/dotnet/aspnetcore/blob/main/docs/EventSourceAndCounters.md. Should give info about what fired a given acceptance/rejection, and perhaps do counters here too. Related to #44140
The text was updated successfully, but these errors were encountered: