summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTomás Touceda <chiiph@leap.se>2013-07-24 13:28:18 -0300
committerTomás Touceda <chiiph@leap.se>2013-07-24 13:57:25 -0300
commit11b3b47ba3ab5ed25b77aab3ce5759c55c9c9627 (patch)
treee545133bebc908b93bad69ef59e2ae84dc40ea8b /src
parentd56790c4cddab8768e14a83a3c9fcb15bda1dee5 (diff)
Escape the domain value to avoid problematic paths
Diffstat (limited to 'src')
-rw-r--r--src/leap/config/providerconfig.py20
-rw-r--r--src/leap/services/eip/providerbootstrapper.py2
2 files changed, 20 insertions, 2 deletions
diff --git a/src/leap/config/providerconfig.py b/src/leap/config/providerconfig.py
index 8b72153a..744199f8 100644
--- a/src/leap/config/providerconfig.py
+++ b/src/leap/config/providerconfig.py
@@ -60,8 +60,26 @@ class ProviderConfig(BaseConfig):
def get_description(self):
return self._safe_get_value("description")
+ @classmethod
+ def sanitize_path_component(cls, component):
+ """
+ If the provider tries to instrument the component of a path
+ that is controlled by them, this will take care of
+ removing/escaping all the necessary elements.
+
+ :param component: Path component to process
+ :type component: unicode or str
+
+ :returns: The path component properly escaped
+ :rtype: unicode or str
+ """
+ # TODO: Fix for windows, names like "aux" or "con" aren't
+ # allowed.
+ return component.replace(os.path.sep, "")
+
def get_domain(self):
- return self._safe_get_value("domain")
+ return ProviderConfig.sanitize_path_component(
+ self._safe_get_value("domain"))
def get_enrollment_policy(self):
"""
diff --git a/src/leap/services/eip/providerbootstrapper.py b/src/leap/services/eip/providerbootstrapper.py
index 274fd3bd..15609620 100644
--- a/src/leap/services/eip/providerbootstrapper.py
+++ b/src/leap/services/eip/providerbootstrapper.py
@@ -179,7 +179,7 @@ class ProviderBootstrapper(AbstractBootstrapper):
"""
leap_assert(domain and len(domain) > 0, "We need a domain!")
- self._domain = domain
+ self._domain = ProviderConfig.sanitize_path_component(domain)
self._download_if_needed = download_if_needed
cb_chain = [