Merge lp:~andreserl/maas/lp1592666 into lp:~maas-committers/maas/trunk

[Andres Rodriguez]

Strip forward slash in the path of the archive mirror, provided that when composing the whole archive, we add the additional "/".

[Gavin Panella (allenap)]

Needs a test!

review: Needs Fixing

[Gavin Panella (allenap)]

review: Approve

=== modified file 'src/maasserver/preseed.py'
--- src/maasserver/preseed.py	2016-07-06 05:37:58 +0000
+++ src/maasserver/preseed.py	2016-07-28 14:06:19 +0000
@@ -573,7 +573,7 @@
     and, optionally, the port.
     """
     parsed_url = urlparse(url)
-    return parsed_url.netloc, parsed_url.path
+    return parsed_url.netloc, parsed_url.path.lstrip("/")
 
 
 def get_preseed_context(osystem='', release='', rack_controller=None):
 
=== modified file 'src/maasserver/tests/test_preseed.py'
--- src/maasserver/tests/test_preseed.py	2016-07-06 05:37:58 +0000
+++ src/maasserver/tests/test_preseed.py	2016-07-28 14:06:19 +0000
@@ -123,12 +123,12 @@
 
     def test_get_netloc_and_path(self):
         input_and_results = [
-            ('http://name.domain:66/my/path', ('name.domain:66', '/my/path')),
-            ('http://name.domain:80/my/path', ('name.domain:80', '/my/path')),
-            ('http://name.domain/my/path', ('name.domain', '/my/path')),
-            ('https://domain/path', ('domain', '/path')),
+            ('http://name.domain:66/my/path', ('name.domain:66', 'my/path')),
+            ('http://name.domain:80/my/path', ('name.domain:80', 'my/path')),
+            ('http://name.domain/my/path', ('name.domain', 'my/path')),
+            ('https://domain/path', ('domain', 'path')),
             ('http://domain:12', ('domain:12', '')),
-            ('http://domain/', ('domain', '/')),
+            ('http://domain/', ('domain', '')),
             ('http://domain', ('domain', '')),
             ]
         inputs = [input for input, _ in input_and_results]
@@ -446,9 +446,9 @@
         self.assertEqual(
             (
                 parsed_main_archive.hostname,
-                parsed_main_archive.path,
+                parsed_main_archive.path.lstrip('/'),
                 parsed_ports_archive.hostname,
-                parsed_ports_archive.path,
+                parsed_ports_archive.path.lstrip('/'),
             ),
             (
                 context['main_archive_hostname'],