[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