404 retrieving javascript asset in dev mode about grails-asset-pipeline

[davydotcom]

ok that needs fixed, in the interum any reason you are requiring knockout js seperately? probably best to have an application.js with

//= require knockout.min.js

//Require your app js

That by chance work at all?

[davydotcom]

Looks like application/x-javascript is an older obsolete equivalent to application/javascript So thats very weird (thanks Safari) :) , Will fix

[gschueler]

that is how i am doing it. However in dev mode, the knockout.min.js is included as a separate <script> tag and so has its own http request

[gschueler]

weird, i tested with chrome too, saw the same format application/x-javascript... is it a Mac thing?

[davydotcom]

yea surprisingly this one hasnt hit me yet... Thanks for opening the issue (Im on a Mac as well so kinda odd)

[davydotcom]

Hmm I just attempted to test this. The mime type is determined by using the servletContext.getMimeType(). I just created a testmin.min.js and it was served properly using grails 2.2.1

[davydotcom]

I assume you are running tomcat in dev correct? Also what jdk version are you running

[gschueler]

I am running Jetty 7.6.0, jdk version 1.7.0_45

[davydotcom]

ah its Jetty then. fantastic.

[davydotcom]

Pushed a release (1.3.0) which should resolve this issue. Asset File definitions can now be defined as multiple accepted content-types.

[davydotcom]

Saw your comment on the commit, whoops! fixed in 1.3.1 wasnt meant to stay.

[gschueler]

👍 thanky

[gschueler]

sorry, i still have the same problem with knockout.min.js

since the format is "application/x-javascript", the uri and extension get changed at this line: https://github.com/bertramdev/asset-pipeline/blob/master/grails-app/controllers/asset/pipeline/AssetsController.groovy#L16

the uri becomes "knockout" and the extension becomes "min"

what is the purpose of that if statement? (line 15)

it seems like this code needs to happen before the if statement:

def contentTypes = AssetHelper.assetMimeTypeForURI(request.forwardURI)
format = contentTypes ? contentTypes[0] : null

[gschueler]

sorry to keep bugging on this issue...i do have a workaround using "-" :)

[davydotcom]

Hah, guess its gonna be one of those days :). its because grails likes to mask off known extensions. i.e. js Hate to ask a silly question, any chance there is a newer jetty version?

[gschueler]

the extension is correct ("js") prior to the if statement

[gschueler]

don't worry about fixing this right away :) if I get a chance I'll write up a unit test for it

[davydotcom]

I finally reproduced it by using Jetty, its definitely a Jetty in dev mode issue. I have a few helpers that have been added to AsseHelper since the original writing of this controller, let me clean it up a bit and this may be a non issue now.

[davydotcom]

Ok, removed the debug println, and built a test app using Jetty and all your versions, finally reproduced the issue and confirmed a fix. Tests are passing for version 1.3.2

[gschueler]

looks good now thanks a bunch! 🍻