[amp-script] Should `text/javascript` be allowed?

[Malvoz]

<amp-script> requires a content-type of application/javascript when src is same-origin, which is documented in https://amp.dev/documentation/components/amp-script/#security-features. Should this be extended to allow text/javascript as well?

https://amp.dev serve scripts with text/javascript, and this HTTP Archive post suggests text/javascript is used widely.

Edit: Also, the HTML spec recommends use of text/javascript for script resources.

[dreamofabear]

Reasonable and a quick change. Interested in contributing a PR? :)

Note that text/javascript is technically marked obsolete.

[adamsilverstein]

Hi @choumx & @Malvoz - Thanks for the bug report, we were seeing this issue as well. I took a first pass at fixing this in #26465.

[dreamofabear]

Fixed by #26465