This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Fix route not found when url is /? #105

Merged

Rich-Harris merged 1 commit into sveltejs:restructure from thgh:patch-2

Feb 3, 2018

Contributor

thgh commented Jan 26, 2018

The regex should always remove the question mark, even when there is nothing behind it.

Alternative code without regex:
req.url.slice(0, (req.url + '?').indexOf('?'))


          Fix route not found when url is /?

422e31e

Rich-Harris merged commit 16cb1fc into sveltejs:restructure

Member

Rich-Harris commented Feb 3, 2018

thanks!

Rich-Harris added a commit that referenced this pull request


          add test for #105

3449f1e

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.