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

text/javascript only #1959

Open
Martii opened this issue May 24, 2022 · 0 comments
Open

text/javascript only #1959

Martii opened this issue May 24, 2022 · 0 comments
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. migration Use this to indicate that it may apply to an existing or announced migration.

Comments

@Martii
Copy link
Member

Martii commented May 24, 2022 •
edited
Loading

What’s missing?

Code point cleanup:

OpenUserJS.org/controllers/user.js

Lines 2034 to 2036 in 673f489

case 'application/x-javascript': // #872 #1661
case 'application/javascript': // #1599
case 'text/javascript': // Default

OpenUserJS.org/controllers/scriptStorage.js

Lines 506 to 509 in 673f489

'text/javascript',
'text/ecmascript',
'application/javascript',
'application/x-javascript',

Why?

Because they've finally standardized it ... at least on paper and not necessarily browser implementation which will need to be fully retested ... to one MIME (Content) type. See also https://www.rfc-editor.org/rfc/rfc9239 and possible next mime-db dep update.

Alternatives you tried

#1661
#1599
#872
@Martii Martii added enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. migration Use this to indicate that it may apply to an existing or announced migration. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. migration Use this to indicate that it may apply to an existing or announced migration.
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Martii