add directory index file support #38

Merged
merged 2 commits into from Jun 26, 2012

2 participants

@aaronmccall

Copied the logic from connect's static middleware which appends 'index.html', if the final character of the file name is '/'.

@tomgco
Owner

If you could provide a test for this I would be happy to pull this in, however If you have problems with the tests let me know and we can sort something out!

@aaronmccall

I'll add a test ASAP. I'm planning on using gzippo in a production site once this feature is merged.

@aaronmccall

@tomgco all the tests are failing with "Error: Response not completed"

Any idea as to why?

@aaronmccall

@tomgco nevermind. It worked after I deleted connect from the node_modules dir.

FYI: The tests don't run properly for me with connect@2.3.3 installed.

@aaronmccall

Rebased against your master and verified that tests do pass with Connect 1.9.0 installed.

@tomgco tomgco merged commit 2b078dd into tomgco:master Jun 26, 2012
@tomgco
Owner

This can now be found in 0.1.7, Thanks!

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