Fix _encode_files detection for __getattr__-based file wrappers

Author: LoicPernod

src/requests/models.py

@@ -236,7 +236,8 @@ def _encode_files(
 
             if isinstance(fp, (str, bytes, bytearray)):
                 fdata = fp
-            elif isinstance(fp, _SupportsRead):  # type: ignore[reportUnnecessaryIsInstance]  # defensive check for untyped callers
+            # data that proxies attributes to underlying objects needs hasattr
+            elif isinstance(fp, _SupportsRead) or hasattr(fp, 'read'):  # type: ignore[reportUnnecessaryIsInstance]  # defensive check for untyped callers
                 fdata = fp.read()
             elif fp is None:  # defensive check for untyped callers
                 continue

tests/test_requests.py

@@ -1087,6 +1087,32 @@ def test_different_encodings_dont_break_post(self, httpbin):
             )
         assert r.status_code == 200
 
+    def test_post_tempfile(self, httpbin):
+        with tempfile.TemporaryFile(mode="w+") as f:
+            f.write("temp file contents\n")
+            f.seek(0)
+            r = requests.post(
+                httpbin("post"),
+                data={"stuff": json.dumps({"a": 123})},
+                params={"blah": "asdf1234"},
+                files={"file": f},
+            )
+        assert r.status_code == 200
+        assert r.json()["files"]["file"] == "temp file contents\n"
+
+    def test_post_named_tempfile(self, httpbin):
+        with tempfile.NamedTemporaryFile(mode="w+") as f:
+            f.write("named temp file contents\n")
+            f.seek(0)
+            r = requests.post(
+                httpbin("post"),
+                data={"stuff": json.dumps({"a": 123})},
+                params={"blah": "asdf1234"},
+                files={"file": f},
+            )
+        assert r.status_code == 200
+        assert r.json()["files"]["file"] == "named temp file contents\n"
+
     @pytest.mark.parametrize(
         "data",
         (