supplied url is altered by patch
Bug #242828 reported by
Robert Collins
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
BzrTools |
New
|
Undecided
|
Unassigned |
Bug Description
14:46 < lifeless> :!bzr patch http://
14:46 < lifeless> bzr: ERROR: http://
permanently redirected to http://
Note that the given url - .../ is requested as ..., and that gets the
redirect because the site wants it to be at .../.
-Rob
affects bzrtools
--
GPG key available at: <http://
To post a comment you must log in.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Robert Collins wrote: paste.ubuntu. com/22781/ plain/ paste.ubuntu. com/22781/ plain is paste.ubuntu. com/22781/ plain/
> Public bug reported:
>
> 14:46 < lifeless> :!bzr patch http://
> 14:46 < lifeless> bzr: ERROR: http://
> permanently redirected to http://
>
> Note that the given url - .../ is requested as ..., and that gets the
> redirect because the site wants it to be at .../.
There are many bzr bugs here:
1. The transport UI doesn't provide a direct way to retrieve a URL; one
must open the base url and then request a file.
2. urlutils.split does not preserve trailing slashes. So splitting paste.ubuntu. com/22781/ plain/ produces paste.ubuntu. com/22781', 'plain')
http://
('http://
3. The transport API does not preserve trailing slashes: paste.ubuntu. com/22781/ plain/' ).get(' ').read() errors. RedirectRequest ed: http:// paste.ubuntu. com/22781/ plain is paste.ubuntu. com/22781/ plain/ paste.ubuntu. com/22781/ ').get( 'plain/ ').read() errors. RedirectRequest ed: http:// paste.ubuntu. com/22781/ plain is paste.ubuntu. com/22781/ plain/
>>> get_transport('http://
bzrlib.
permanently redirected to http://
>>> get_transport('http://
bzrlib.
permanently redirected to http://
Aaron enigmail. mozdev. org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iD8DBQFIYdQH0F+ nu1YWqI0RAqy5AJ 97Svp7G8Mtk9/ Wzxwk5nyOIkXsrA CcD9aQ U8bb6jvM=
cUV6E9cso9wTIn1
=8juX
-----END PGP SIGNATURE-----