Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Should Content-Type: application/javascript also include a 'charset=...'? #396

Open
etnt opened this issue Nov 19, 2019 · 1 comment
Open

Comments

@etnt
Copy link
Collaborator

etnt commented Nov 19, 2019

We are running some security scanning tools that are complaining about this.
So my question is if perhaps a 'charset=' should be appended to this content type?
(the obsoleted 'text/javascript' would of course get a charset since it falls within 'text/*')

@vinoski
Copy link
Collaborator

vinoski commented Nov 20, 2019

I think that's OK, but I'll need to look into the issue more.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants