jongleberry comments

Results 313 comments of


                                            jongleberry

Should it have a folder view when it doesn't have an index.html?

this module doesn't support directories

don't read to file when the browser cache is valid

yes, I try not to serve files directly on my node.js server. they should be served via CDN. However, I understand how the developer UX for this modular is subpar....

Teach route to squeeze slashes

I understand the intention here, but it should be optional through a flag

Don't throw an exception in case if invalid path escape

can you write a test? or share what the test case is

dethroy -> destroy!

yup :D

Newoptions

Can't tell what the diff is. Can you undo spacing styles?

add noStackEnvs option

yeah this implementation is over complicated. and `process.env.NODE_ENV.toLowerCase()` is going to fail if `NODE_ENV` doesn't happen to be defined.

make this library actually work

had trouble with using the compiled binary on Heroku. I just compile from source now and it works: https://github.com/funraiseme/webp

add stats and graphs

how are you doing this? should we add analytics to `component(1)`?

investigate ways of shrinking the filesize

+1 i love rewrites