Sapper: redirect won't go to `/` but empty quotes works

[arxpoetica]

(In preload) like so:

return this.redirect(302, '/') // it be broke
return this.redirect(302, '') // it work

[arxpoetica]

Probably another way to say this, leading slashes don't work.

[Rich-Harris]

How do you think this ought to be handled? Should Sapper just strip any leading / character?

[arxpoetica]

Good Q. Maybe follow Express conventions? From https://expressjs.com/en/guide/routing.html:

Express uses path-to-regexp for matching the route paths; see the path-to-regexp documentation for all the possibilities in defining route paths.

It appears the leading / is optional: /^\/?$/i

Update Q: is routing a separate issue though?

[s3ththompson]

This threw me today. I expected the routes to be relative if they didn't have a starting /...

[Rich-Harris]

released 0.15.7 with the fix