sftp bugs

[Jan Hudec]

On Wed, Nov 30, 2005 at 09:33:36 +0800, James Henstridge wrote:
> Matthieu Moy wrote:
> >[...]
> >This is because _parse_url does a
> >
> >            # remove leading '/'
> >            self._path = urllib.unquote(self._path[1:])
> >
> >(shouldn't this be a urllib.unquote(self._path.lstrip('/')) to remove
> >multiple slashes?)
> >  
> >
> I think the URL handling fixes in the patch I posted yesterday should
> fix this problem.
> 
> Removing multiple slashes is not necessarily correct though, since the
> SFTP URI spec is a bit weird: "sftp://localhost/tmp" refers to "~/tmp",
> while "sftp://localhost/%2Ftmp" refers to "/tmp".
> 
> It doesn't say how "sftp://localhost//tmp" should be interpreted

If I read it correctly, it said it's illegal.

> though.  Stripping one slash will give you "/tmp", while stripping all
> slashes gives you "~/tmp".

/tmp seems more useful, since it simplifies typing the absolute paths a
bit.

--
						 Jan 'Bulb' Hudec <bulb at ucw.cz>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051130/67fe151c/attachment.pgp