Fixes directory middleware for windows paths #645

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants

stagas commented Aug 18, 2012

No description provided.

@tj tj commented on the diff Aug 18, 2012

lib/middleware/directory.js
@@ -174,7 +174,7 @@ function html(files, dir, useIcons) {
}
return '<li><a href="'
- + join(dir, file)
+ + slashes(join(dir, file))
@tj

tj Aug 18, 2012

Member

my bad, we should just use dir + '/' + file here

@dougwilson

dougwilson Oct 15, 2013

Contributor

dir + '/' + file does not work, because when dir is / it results in //test for file = 'test'.

Any chance this will be added soon?

Member

tj commented Oct 29, 2012

that one line needs changing

Contributor

jonathanong commented Sep 24, 2013

@stagas bump

jonathanong closed this in #914 Oct 15, 2013

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