Revert "Fix serving files with diacritics in name under their original name"

This reverts commit 1741fd9adf70800fd692c24b0d85f5caad6a44e5.

1 files changed, 8 insertions, 26 deletions

diff --git a/cherrypy/lib/static.py b/cherrypy/lib/static.py
index b255a6b4..9a3b8e83 100644
--- a/cherrypy/lib/static.py
+++ b/cherrypy/lib/static.py
@@ -28,22 +28,6 @@ def _setup_mimetypes():
 _setup_mimetypes()
 
 
-def _utf8_content_disposition(disposition, file_name):
-    """Create HTTP header for downloading a file with UTF-8 filename.
-
-    See this and related answers:
-    https://stackoverflow.com/a/8996249/2173868
-    """
-    ascii_name = (
-        unicodedata.normalize('NFKD', file_name).
-        encode('ascii', errors='ignore').decode()
-    )
-    quoted_name = urllib.parse.quote(file_name)
-    header = u'{}; filename="{}"'.format(disposition, ascii_name)
-    header += u'; filename*=UTF-8\'\'{}'.format(quoted_name)
-    return header
-
-
 def serve_file(path, content_type=None, disposition=None, name=None,
                debug=False):
     """Set status, headers, and body in order to serve the given path.
@@ -53,10 +37,9 @@ def serve_file(path, content_type=None, disposition=None, name=None,
     of the 'path' argument.
 
     If disposition is not None, the Content-Disposition header will be set
-    to "<disposition>; filename=<name>; filename*=utf-8''<name>"
-    as described in RFC6266 (https://tools.ietf.org/html/rfc6266#appendix-D).
-    If name is None, it will be set to the basename of path.
-    If disposition is None, no Content-Disposition header will be written.
+    to "<disposition>; filename=<name>". If name is None, it will be set
+    to the basename of path. If disposition is None, no Content-Disposition
+    header will be written.
     """
     response = cherrypy.serving.response
 
@@ -109,7 +92,7 @@ def serve_file(path, content_type=None, disposition=None, name=None,
     if disposition is not None:
         if name is None:
             name = os.path.basename(path)
-        cd = _utf8_content_disposition(disposition, name)
+        cd = '%s; filename="%s"' % (disposition, name)
         response.headers['Content-Disposition'] = cd
     if debug:
         cherrypy.log('Content-Disposition: %r' % cd, 'TOOLS.STATIC')
@@ -128,10 +111,9 @@ def serve_fileobj(fileobj, content_type=None, disposition=None, name=None,
     The Content-Type header will be set to the content_type arg, if provided.
 
     If disposition is not None, the Content-Disposition header will be set
-    to "<disposition>; filename=<name>; filename*=utf-8''<name>"
-    as described in RFC6266 (https://tools.ietf.org/html/rfc6266#appendix-D).
-    If name is None, 'filename' will not be set.
-    If disposition is None, no Content-Disposition header will be written.
+    to "<disposition>; filename=<name>". If name is None, 'filename' will
+    not be set. If disposition is None, no Content-Disposition header will
+    be written.
 
     CAUTION: If the request contains a 'Range' header, one or more seek()s will
     be performed on the file object.  This may cause undesired behavior if
@@ -167,7 +149,7 @@ def serve_fileobj(fileobj, content_type=None, disposition=None, name=None,
         if name is None:
             cd = disposition
         else:
-            cd = _utf8_content_disposition(disposition, name)
+            cd = '%s; filename="%s"' % (disposition, name)
         response.headers['Content-Disposition'] = cd
     if debug:
         cherrypy.log('Content-Disposition: %r' % cd, 'TOOLS.STATIC')