Version 2.3.7 broke connect.static for me #629

Closed
rbackhouse opened this Issue Jul 12, 2012 · 3 comments

Comments

Projects
None yet
2 participants
@rbackhouse

Moving to 2.3.7 has resulted in code like :

var app = connect()
.use("/_javascript", connectOptimizer)
.use(connect.static(appdir))
.use(connect.static(zazloptimizer.getLoaderDir()));

i.e with more that one use of connect.static

failing to pass on to the 2nd connect.static if a resource is not found in the first one.

I get these reported to the console :

Error: ENOENT, stat '/Users/rbackhouse/dev/zazldev/test/zazlnodejs/examples/loader/amd/zazl.js'

Version 2.3.6 works fine with this code

@tj

This comment has been minimized.

Show comment
Hide comment
@tj

tj Jul 12, 2012

Member

i must have overlooked some 404 handling, I'll check it out and add test cov

Member

tj commented Jul 12, 2012

i must have overlooked some 404 handling, I'll check it out and add test cov

@tj tj closed this in 8daa67e Jul 12, 2012

@tj

This comment has been minimized.

Show comment
Hide comment
@tj

tj Jul 12, 2012

Member

thanks for the report

Member

tj commented Jul 12, 2012

thanks for the report

@rbackhouse

This comment has been minimized.

Show comment
Hide comment
@rbackhouse

rbackhouse Jul 12, 2012

np. thanks for the quick fix

np. thanks for the quick fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment