[PATCH] Allow to download public bundle as mbox

Jeremy Kerr jk at ozlabs.org
Wed Nov 7 02:19:46 EST 2012


Hi Simo,

>> In other words, does anyone rely on the bundle name to contain the ID?
>
> Note that I didn't really change it, for normal mboxes the name is
> str(bundle.id) :-)

Yep, but if we unify them, it'll be more consistent. Using the name 
seems to be the way to go.

(however, we should also clean the filename, so that we don't get 
user-provided data ending up in a HTTP header field)

Cheers,


Jeremy



More information about the Patchwork mailing list