Skip to content

Commit 8cbc617

Browse files
committed
Make requests.Session instance var private
1 parent d9e5ced commit 8cbc617

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pyiceberg/io/fsspec.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,11 +113,11 @@ def __call__(self, request: "AWSRequest", **_: Any) -> None:
113113
class S3V4RestSigner(S3RequestSigner):
114114
"""An S3 request signer that uses an external REST signing service to sign requests."""
115115

116-
session: requests.Session
116+
_session: requests.Session
117117

118118
def __init__(self, properties: Properties) -> None:
119119
super().__init__(properties)
120-
self.session = requests.Session()
120+
self._session = requests.Session()
121121

122122
def __call__(self, request: "AWSRequest", **_: Any) -> None:
123123
signer_url = self.properties.get(S3_SIGNER_URI, self.properties[URI]).rstrip("/") # type: ignore
@@ -135,7 +135,7 @@ def __call__(self, request: "AWSRequest", **_: Any) -> None:
135135
"headers": {key: [val] for key, val in request.headers.items()},
136136
}
137137

138-
response = self.session.post(
138+
response = self._session.post(
139139
f"{signer_url}/{signer_endpoint.strip()}", headers=signer_headers, json=signer_body
140140
)
141141
try:

0 commit comments

Comments
 (0)