chore(filer): url quote content disposition

Author: duanhongyi

rootfs/api/files/parsers.py

@@ -19,27 +19,40 @@ class FilerUploadParser(FileUploadParser):
     """
     media_type = 'filer/octet-stream'
 
+    errors = {
+        "no_filepath": (
+            "Missing filepath. Request should include a Content-Disposition "
+            "header with a filepath parameter."
+        ),
+    }
+    errors.update(FileUploadParser.errors)
+
     def parse(self, stream, media_type=None, parser_context=None):
         request = parser_context['request']
         filename = self.get_filename(stream, media_type, parser_context)
         if not filename:
             raise ParseError(self.errors['no_filename'])
-
+        filepath = self.get_filepath(stream, media_type, parser_context)
+        if not filepath:
+            raise ParseError(self.errors['no_filepath'])
         try:
             content_length = int(request.META.get('HTTP_CONTENT_LENGTH',
                                                   request.META.get('CONTENT_LENGTH', 0)))
         except (ValueError, TypeError):
             content_length = None
+        return DataAndFiles({}, {'file': FilerFile(filename, filepath, stream, content_length)})
 
-        file_meta = self.get_file_meta(request.META)
-        return DataAndFiles({}, {'file': FilerFile(
-            file_meta['filename'], file_meta['filepath'], stream, content_length)})
-
-    def get_file_meta(self, META):
+    def get_filepath(self, stream, media_type, parser_context):
         """
-        Detects the uploaded file name. First searches a 'filename' url kwarg.
+        Detects the uploaded file name. First searches a 'filepath' url kwarg.
         Then tries to parse Content-Disposition header.
         """
+        with contextlib.suppress(KeyError):
+            return parser_context['kwargs']['filepath']
+
         with contextlib.suppress(AttributeError, KeyError, ValueError):
-            _, params = parse_header_parameters(META['HTTP_CONTENT_DISPOSITION'])
-            return params
+            meta = parser_context['request'].META
+            _, params = parse_header_parameters(meta['HTTP_CONTENT_DISPOSITION'])
+            if 'filepath*' in params:
+                return params['filepath*']
+            return params['filepath']