Skip to content
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.

Sign up for GitHub

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

Jump to bottom

null check missing for httpcontext.response.contenttype #401

Closed
scoroop opened this issue May 4, 2020 · 1 comment
Closed

null check missing for httpcontext.response.contenttype #401

scoroop opened this issue May 4, 2020 · 1 comment

Comments

@scoroop
Copy link

scoroop commented May 4, 2020

i18n/src/i18n/LocalizedApplication.cs

Line 337 in 395ff75

&& Current.ContentTypesToLocalize.Match(context.Response.ContentType).Success) // Include certain content types from being processed

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
Copy link
Contributor

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

@vhatuncev vhatuncev mentioned this issue Sep 5, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@turquoiseowl @vhatuncev @scoroop