From 6e310790334b2cd7f7ba72cf3dbd3585176ccbdf Mon Sep 17 00:00:00 2001 From: Joe Kilner Date: Thu, 27 Oct 2016 15:53:53 -0700 Subject: [PATCH] Add sso to list of known schemes for relative URLs repo already special-cases sso:// URLs to behave similarly to https:// and rpc:// elsewhere in repo, but it forgot to do so here. Noticed when trying to use relative URLs in a manifest obtained using an sso:// URL. Change-Id: Ia11469a09bbd6e444dbc4f22c82f9bbe9f5fd083 --- manifest_xml.py | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/manifest_xml.py b/manifest_xml.py index 9c882af6..0859e1fb 100644 --- a/manifest_xml.py +++ b/manifest_xml.py @@ -40,8 +40,18 @@ LOCAL_MANIFEST_NAME = 'local_manifest.xml' LOCAL_MANIFESTS_DIR_NAME = 'local_manifests' # urljoin gets confused if the scheme is not known. -urllib.parse.uses_relative.extend(['ssh', 'git', 'persistent-https', 'rpc']) -urllib.parse.uses_netloc.extend(['ssh', 'git', 'persistent-https', 'rpc']) +urllib.parse.uses_relative.extend([ + 'ssh', + 'git', + 'persistent-https', + 'sso', + 'rpc']) +urllib.parse.uses_netloc.extend([ + 'ssh', + 'git', + 'persistent-https', + 'sso', + 'rpc']) class _Default(object): """Project defaults within the manifest."""