Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign up
What is the current behavior?
Three of the tests related to homepage calculation. The are all of the following flavor.
and here is a sample of the HTML being returned from
get. As you can see, Windows path separators () are being used indicating thatpath.joinis likely involved. I've not isolated the particular code as yet.What changes are you suggesting?
Suggest we do an audit of all the places that inject URLs into content or headers and ensure they don't use
path.join. In fact, we should make a helper (e.g.,safeJoin) that does something like(...segments) => slash(path.join(segments))and then use that everywhere that has anything to do with web results.cc @github/docs-engineering , @chiedo