h5bp / server-configs-iis

IIS Web.Config Boilerplates
MIT License
336 stars 85 forks source link

Change existingResponse from PassThrough to Auto #30

Closed rexcfnghk closed 8 years ago

rexcfnghk commented 8 years ago

In ASP.NET MVC, PassThrough will serve a blank page when we serve a >4XX response (e.g. using return new HttpStatusCode(HttpStatusCode.Forbidden)) even when a route is set for 403 errors. By reverting to Auto, the view will be correctly rendered.

rexcfnghk commented 8 years ago

Will open the PR again once I rebased my commits

ChrisMcKee commented 8 years ago

Cool; slow reply I know. Ideally we want to add a comment above the line with the parameter in and why you might remove it as it helps people make their own choices / debug their own issues along the way. Try and avoid whitespace changes if you can :)

Thanks

rexcfnghk commented 8 years ago

The new PR is on #31 :+1: Let me tidy up the commit and remove the whitespaces. Thanks.