Merge pull request #556 from inkhey/fix/550/fix_root_workspace_file_webdav

tracim/tracim/lib/webdav/sql_resources.py

@@ -160,6 +160,10 @@ class Workspace(DAVCollection):
 
         self._file_count = 0
 
+        # FIXME : Remove this regex when tracim become
+        # able to deal with file at root of workspace
+        self._subfolder_regex = re.compile('^{}[^/]*$'.format(environ['http_authenticator.realm']))  # nopep8
+
     def __repr__(self) -> str:
         return "<DAVCollection: Workspace (%d)>" % self.workspace.workspace_id
 
@@ -215,6 +219,12 @@ class Workspace(DAVCollection):
         if resource:
             content = resource.content
 
+        # FIXME : Remove this regex when tracim become
+        # able to deal with file at root of workspace
+        if self._subfolder_regex.match(dirname(path)):
+            raise DAVError(HTTP_FORBIDDEN)
+
+
         return FakeFileStream(
             file_name=file_name,
             content_api=self.content_api,
@@ -399,6 +409,7 @@ class Folder(Workspace):
             invalid_path = invalid_path or any(x in self.path for x in ['.deleted', '.archived'])
             invalid_path = invalid_path or dirname(destpath) == self.environ['http_authenticator.realm']
 
+
             if not invalid_path:
                 self.move_folder(destpath)
 
@@ -833,6 +844,9 @@ class File(DAVNonCollection):
         self.content = content
         self.user = UserApi(None).get_one_by_email(environ['http_authenticator.username'])
         self.content_api = ContentApi(self.user)
+        # FIXME : Remove this regex when tracim become
+        # able to deal with file at root of workspace
+        self._subfolder_regex = re.compile('^{}[^/]*$'.format(environ['http_authenticator.realm']))  # nopep8
 
         # this is the property that windows client except to check if the file is read-write or read-only,
         # but i wasn't able to set this property so you'll have to look into it >.>
@@ -915,6 +929,9 @@ class File(DAVNonCollection):
             invalid_path = any(x in destpath for x in ['.deleted', '.archived'])
             invalid_path = invalid_path or any(x in self.path for x in ['.deleted', '.archived'])
             invalid_path = invalid_path or dirname(destpath) == self.environ['http_authenticator.realm']
+            # FIXME : Remove this regex when tracim become
+            # able to deal with file at root of workspace
+            invalid_path = invalid_path or self._subfolder_regex.match(dirname(destpath))
 
             if not invalid_path:
                 self.move_file(destpath)