Comment 1 for bug 728209

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 3/3/2011 5:14 AM, Robert Collins wrote:
> http://bazaar.launchpad.net/%7Edaker/dexter-
> rolodex/fix.688511/view/24.1.5/data/media/background.png?sort=filename&remember=1&file_id=etc-20101119044251-5jfdakonern1nd08-2&compare_revid=1

Going here:
http://bazaar.launchpad.net/~daker/dexter-rolodex/fix.688511/view/24.1.5/data/media/background.png

Shows that it is a binary file.

I think the issue is that "compare_revid" is confusing us.

Going here:
http://bazaar.launchpad.net/~daker/dexter-rolodex/fix.688511/view/1/data/media/background.png

seems to also say that it is a binary file. But given the
'compare_revid', it looks like it wants the '1' to be a revision id, not
a revno.

Though expanding the 1 to the actual revision_id still gives an oops:
OOPS-1888CBB49

The fact that it is also passing 'etc-...' as the file_id while also
having media/background.png seems pretty confusing.

It certainly looks like someone hand-crafted that url. Regardless, we
shouldn't be OOPSing.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk1vV+gACgkQJdeBCYSNAANF6ACdG6pgJspVFNjmihQiR32uIboy
64MAoIASGq9vpjdDGmLffxFFaqdv0P5y
=oFvF
-----END PGP SIGNATURE-----