turquoiseowl / i18n

Smart internationalization for ASP.NET
Other
556 stars 156 forks source link

null check missing for httpcontext.response.contenttype #401

Closed scoroop closed 3 years ago

scoroop commented 4 years ago

https://github.com/turquoiseowl/i18n/blob/395ff751e1c1cbfce396b058e21aa44674dc1bc3/src/i18n/LocalizedApplication.cs#L337

I'm working on legacy app and it turns out it sometimes has null value in contenttype of response. Null-check would make sense here - at the moment as a workaround I just set it explicitly to default value

vhatuncev commented 4 years ago

Bug has been fixed already in this PR, but still not merged 😞