search

proudmonkey / AutoWrapper

A simple, yet customizable global exception handler and Http response wrapper for ASP.NET Core APIs.
MIT License
677 stars 82 forks source link

Fix using `AutoWrapIgnore` is still wrapper #56

Closed hueifeng closed 4 years ago

hueifeng commented 4 years ago

Fix using AutoWrapIgnore is still wrapper #55

proudmonkey commented 4 years ago

@hueifeng Thank you for the PR. Have you run some test scenarios for this?

hueifeng commented 4 years ago

@proudmonkey I tested it and it works, because when we set ignore, there is no need to wrap the result set anymore. You can directly return the results.

proudmonkey commented 4 years ago

@hueifeng actually it will return an empty response since you are not writing back the to the context.