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.

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

Caching application/x-javascript #71

Closed
adeelejaz opened this issue Aug 23, 2010 · 7 comments
Closed

Caching application/x-javascript #71

adeelejaz opened this issue Aug 23, 2010 · 7 comments

Comments

@adeelejaz
Copy link
Contributor

application/x-javascript has been added to sent as gzip'ed. Shouldn't application/x-javascript mime type be also cached with the rest?

@adeelejaz
Copy link
Contributor Author

also, image/gif?

@paulirish
Copy link
Member

gif! haha.
okay fine fine. :)

do you want to fork and commit that? i can get around to it if you cant

@adeelejaz
Copy link
Contributor Author

Just want to make sure we have all the bases covered ;)

Forking.. I was looking for a reason to fork this anyway!

@adeelejaz
Copy link
Contributor Author

Ok.. some digging around and reading has brought something else up..

The Network Working Group published a memo in 2006, calling the following media types obsolete (along with a few others):
text/javascript
text/ecmascript
text/x-javascript
text/x-ecmascript
application/x-javascript
application/x-ecmascript
Source: Scripting Media Types (http://www.ietf.org/rfc/rfc4329.txt)

The document suggests the following to be used instead:
application/javascript
application/ecmascript

Oddly enough the W3C allows all of them (backward compatibility I guess):
http://dev.w3.org/html5/spec/Overview.html#scriptingLanguages

So much for standards huh.. I've pushed the changes btw.

@paulirish
Copy link
Member

we can close this right, adeelejaz?

holler at me if this is still an open issue.

@adeelejaz
Copy link
Contributor Author

My only concern is 'application/x-javascript' is sent via gzip but not given an expiry header. Shouldn't we just remove 'application/x-javascript' all together or just add an expiry header for it to keep things consistent?

@paulirish
Copy link
Member

Okay sure. Let's remove x-javascript from the gzip line.
if you commit that i'll pull it in.

mdunhem pushed a commit to mdunhem/html5-boilerplate that referenced this issue Jun 6, 2012
lasharela pushed a commit to lasharela/sashkas-book-life-and-beyond that referenced this issue Aug 6, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants