-
Notifications
You must be signed in to change notification settings - Fork 61
Double Slashes at the start of the path when redirecting to an absolute url #177
Comments
…a path when redirecting to an absolute url
Fab, thanks. I'll try and take a look during/after Christmas, ensure everything works okay still and deploy in the New Year :) |
No worries, I'm having to use a custom build for now. |
@ProNotion I've tried to replicate this before pulling in your request, but I can't replicate it to ensure your fix will work. Perhaps I'm missing a step? I've done the following Upon save, the domain gets removed from the old URL, converting it to a relative URL. |
Hi Daniel, Sorry I should have been clearer. The absolute url would be for an external address. So if my site domain was http://www.example.com/ and my old url was |
@daniel-chenery Try leaving the trailing slash off of your redirect url and adding a querystring value something like this: |
…d slash or it breaks urls
When creating a redirect to an absolute url e.g
/old-url
tohttp://example.com/new-url
you end up with the path beginning with//
and so get a final url ofhttp://example.com//new-url
which causes issues.I've patched it and will submit a pull request shortly.
The text was updated successfully, but these errors were encountered: