← Back to team overview

launchpad-dev team mailing list archive

Re: Handling of file:/// urls in the database

 

Jelmer Vernooij wrote:
> On Thu, 2009-11-26 at 19:30 +1300, Michael Hudson wrote: 
>> Stuart Bishop wrote:
>>> On Thu, Nov 26, 2009 at 11:08 AM, Michael Hudson
>>> <michael.hudson@xxxxxxxxxxxxx> wrote:
>>>> Something that turns out to a bit annoying when you try to test bzr-svn
>>>> with Launchpad is that bzr doesn't allow a netloc part in file:// urls
>>>> and the launchpad "valid_absolute_url" insists on a netloc in all URLs
>>>> (of course it's essentially always 'localhost' in file:// URLs).  This
>>>> is a bit stupid for bzr and I'll fix it to accept file://localhost/
>>>> URLs, but would it be possible to change this for Launchpad too?
>>> I would think file: URLs are one of the things that valid_absolute_url
>>> is supposed to catch, as on the production system it would certainly
>>> indicate a mistake or an attack (the database constraint is our second
>>> layer of defense after the form validation).
>> Well, we more-or-less need it for testing bzr-svn (and other) imports.
>> I don't really want to spin up an apache with the mod_dav_svn installed
>> for the test...  We've also used file://localhost/ urls to do imports
>> from disk in the past, although I think there are probably better ways
>> of doing this.
> You should be able to start svnserve locally and test using svn://.

Thanks for that idea, it worked nicely.  Seems like a more realistic
test too, which is a nice bonus!

Cheers,
mwh



References