v2.0.1 Adds extra slash to urls with a capital "A" followed by a "/" in them
See original GitHub issueUrls with a capital “A” followed by a slash are getting an extra slash appended. This doesn’t appear to be the case with other capital letters or lower case “a”.
This doesn’t appear to be an issue in 2.0.0.
const join = require('url-join')
// correct
join('http://example.com', 'A')
// http://example.com/A
// correct
join('http://example.com', 'A/')
// http://example.com/A/, correct
join('http://example.com/', 'A/', '/')
// http://example.com/A//
join('http://example.com', 'A', '/')
// http://example.com/A//
join('http://example.comA/', 'hello')
// http://example.comA//hello
join('http://example.comA', '/hello')
// http://example.comA//hello
Issue Analytics
- State:
- Created 6 years ago
- Reactions:1
- Comments:5 (2 by maintainers)
Top Results From Across the Web
mod_alias redirects adding two trailing slashes - Stack Overflow
When users type these in with a trailing slash, such as http:/example.com/folder/vanity1/ they redirect to a URL with two trailing slashes, such ...
Read more >Trailing URL Slashes in Django - LearnDjango.com
Note: The Chrome web browser automatically appends a trailing slash / to URLs so you'll need to use something like Safari to see...
Read more >Remove Double Slashes in URL - Citrix Discussions
Hi Guys, I would like to use a responder or rewrite policy to remove unnecessary double slashes in URLs, ...
Read more >Should You Have a Trailing Slash at the End of URLs? - Ahrefs
A trailing slash is a forward slash (“/”) placed at the end of a URL such as domain.com/ or domain.com/page/. But should you...
Read more >RFC 3986: Uniform Resource Identifier (URI): Generic Syntax
1. Base URI Embedded in Content Within certain media types, a base URI for relative references can be embedded within the content itself...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
PR is there https://github.com/jfromaniello/url-join/pull/26
I just merged #26 and published as
v2.0.2
. Thanks everyone for reporting, confirming and fixing!I am deeply sorry I took so much time to merge and release.