Merge pull request #2495 from oefe/progress-wrap_file-add-mode

Fix missing `mode` property on file wrapper

Author: willmcgugan

CHANGELOG.md

@@ -19,6 +19,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Handle stdout/stderr being null https://github.com/Textualize/rich/pull/2513
 - Fix NO_COLOR support on legacy Windows https://github.com/Textualize/rich/pull/2458
+- Fix missing `mode` property on file wrapper breaking uploads via `requests` https://github.com/Textualize/rich/pull/2495
 
 ## [12.5.2] - 2022-07-18
 

CONTRIBUTORS.md

@@ -28,6 +28,7 @@ The following people have contributed to the development of Rich:
 - [Paul McGuire](https://github.com/ptmcg)
 - [Antony Milne](https://github.com/AntonyMilneQB)
 - [Michael Milton](https://github.com/multimeric)
+- [Martina Oefelein](https://github.com/oefe)
 - [Nathan Page](https://github.com/nathanrpage97)
 - [Avi Perl](https://github.com/avi-perl)
 - [Laurent Peuch](https://github.com/psycojoker)

rich/progress.py

@@ -216,6 +216,10 @@ def fileno(self) -> int:
     def isatty(self) -> bool:
         return self.handle.isatty()
 
+    @property
+    def mode(self) -> str:
+        return self.handle.mode
+
     @property
     def name(self) -> str:
         return self.handle.name

tests/test_progress.py

@@ -614,6 +614,7 @@ def test_wrap_file() -> None:
         with open(filename, "rb") as file:
             with rich.progress.wrap_file(file, total=total) as f:
                 assert f.read() == b"Hello, World!"
+                assert f.mode == "rb"
                 assert f.name == filename
             assert f.closed
             assert not f.handle.closed