Duplicate slashes "//" in urls when a part is empty

url-join version 3.0.0

> urlJoin('path/', '', '/test')
'path/test'  # expected
'path//test' # actual

> urlJoin('path', '', /test')
'path/test'  # expected
'path//test' # actual

> urlJoin('path', null, /test')
'path/test'  # expected
'path//test' # actual

In more detail, I’ve got an array of bits of a url. Sometimes, parts are optional, so I set them to ‘’ and assume the url joiner will throw away the empty bits

[crccheck]

ooh, I didn’t even think of that use case. I’ll try to keep that in mind in the future too. For reference, here’s Node’s path.join with some variations of that case:

> path.join('', 'foo.js')
'foo.js'
> path.join('', '/foo.js')
'/foo.js'
> path.join('/', 'foo.js')
'/foo.js'
> path.join('/', '/foo.js')
'/foo.js'

[jfromaniello]

Thanks for reporting this. I just published v4 addressing this the same way that path.join does.