-
Notifications
You must be signed in to change notification settings - Fork 25.2k
Update filters.md #35627
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
Update filters.md #35627
Conversation
Remove a problematic image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the filters documentation to fix issue #35598 by removing an incorrect diagram and expanding on the details of exception filters.
- Removes the outdated filter pipeline diagram.
- Updates the exception filters section to include additional execution details.
@@ -65,18 +65,18 @@ Each filter type is executed at a different stage in the filter pipeline: | |||
* Are **not** supported in Razor Pages. | |||
* Can be invoked on both actions and route handler-based endpoints. | |||
|
|||
* [Exception filters](#exception-filters) apply global policies to unhandled exceptions that occur before the response body has been written to. | |||
* [Exception filters](#exception-filters): | |||
* Apply global policies to unhandled exceptions that occur before the response body has been written to. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Apply global policies to unhandled exceptions that occur before the response body has been written to. | |
* Apply global policies to unhandled exceptions that occur before the response body has been written to. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Just a very minor misalignment on a list, I removed the extra space.
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Co-authored-by: Wade Pickett <wpickett@microsoft.com>
Pull Request is not mergeable
Pull Request is not mergeable
Pull Request is not mergeable
Fixes #35598
Remove an incorrect diagram.
Internal previews