Restore support for using pathlib.Path for static_folder.

* No longer causes AttributeError: 'PosixPath' object has no
  attribute 'rstrip'.

* This was broken by e6178fe
  which was released in 1.1.2.

* Add a regression test that now passes.

* Only rstrip os.path.sep from the static_folder value,
  rather than always rstripping both / and \ regardless of os.

See #3557.

Author: jab

CHANGES.rst

@@ -1,5 +1,14 @@
 .. currentmodule:: flask
 
+Version 1.1.x
+-------------
+
+Not yet released.
+
+-   Officially support passing a :class:`pathlib.Path` for
+    ``static_folder`` which stopped working in 1.1.2.
+
+
 Version 1.1.2
 -------------
 

src/flask/helpers.py

@@ -1001,7 +1001,7 @@ def static_folder(self):
     @static_folder.setter
     def static_folder(self, value):
         if value is not None:
-            value = value.rstrip("/\\")
+            value = fspath(value).rstrip(os.path.sep)
         self._static_folder = value
 
     @property

tests/test_basic.py

@@ -1425,6 +1425,16 @@ def test_static_url_empty_path_default(app):
     rv.close()
 
 
+@pytest.mark.skipif(sys.version_info < (3, 6), reason="requires Python >= 3.6")
+def test_static_folder_with_pathlib_path(app):
+    from pathlib import Path
+
+    app = flask.Flask(__name__, static_folder=Path("static"))
+    rv = app.test_client().open("/static/index.html", method="GET")
+    assert rv.status_code == 200
+    rv.close()
+
+
 def test_static_folder_with_ending_slash():
     app = flask.Flask(__name__, static_folder="static/")