3 ms·
Link needs to be edited (has an extra trailing /)
by _aavaa_ 5y ago
Link needs to be edited (has an extra trailing /)
- Jtsummers 5y ago<link data-react-helmet="true" rel="canonical" href="https://automerge.github.io//"> From the site itself. Interestingly, if you turn off JS the page loads just fine as is. Their own canonical link breaks with their own Javascript.
- gumby 5y agoReading the URL spec, the path elements (separated by /) are allows to be void, so // is syntactically valid. And POSIX specifies that // should be treated as / . I know that these days that they are rarely actually file references, but still I feel that web servers should accept this, especially a site like github.
- Jtsummers 5y agoJust tried it out, Github does, in fact, let you use extra slashes. I just threw a bunch of /'s onto https://github.com/////// https://github.com/////// and it loaded just fine. The problem is with the JS on this particular page. When it first loads (on my slower and older laptop there was enough delay to see this) it loads the Automerge home page just fine. Then it tries to pull in contents because of that extra / and breaks. If you go to the Docusaurus home page (which is what Automerge's home page is built with) you can get similar behavior). https://docusaurus.io/ https://docusaurus.io/ https://docusaurus.io// https://docusaurus.io// Try the second with JS turned off as well as on, you will get the home page when it's off and a 404 when it's on. So it seems the problem is two-fold: 1. Automerge has the wrong canonical link, which HN pulls from submissions and replaces submitted URLs with (this is, I think, largely to help with de-duping). 2. Docusaurus' JS (which Automerge is using) takes a perfectly valid URL and generates a 404.