search

NickCraver / StackExchange.Exceptional

Error handler used for the Stack Exchange network
https://nickcraver.com/StackExchange.Exceptional/
Apache License 2.0
863 stars 171 forks source link

Capture data on System.Web.HttpException exceptions. Fixes #147 #148

Closed seankearney closed 5 years ago

seankearney commented 6 years ago

System.Web.HttpException exceptions weren't being captured with all available data (such as Url)

NickCraver commented 6 years ago

@seankearney Unfortunately, this regresses on https://github.com/NickCraver/StackExchange.Exceptional/commit/2fad0bc3fac2d121a58a6a870a2609f85b360ae3 which was explicitly added to handle further null refs during startup conditions. I'd like to solve the issue...but we'll need to find another approach here.

Side note: for 2 PRs, you'll need 2 branches, as pushing to an existing branch will update every PR with it (as happened here).

NickCraver commented 5 years ago

With no updates or changes here - going to close this out given the problems above. An issue explaining exactly what's happening (the scenario) would be very helpful.