Skip to content

Commit c029cfd

Browse files
authored
Merge envars before sending a request (#729)
Taken from the `requests` docs: https://requests.readthedocs.io/en/latest/user/advanced/#prepared-requests Fixes #727
1 parent d209f25 commit c029cfd

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

kagglesdk/kaggle_http_client.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,9 @@ def call(
119119
self._init_session()
120120
http_request = self._prepare_request(service_name, request_name, request)
121121

122-
http_response = self._session.send(http_request)
122+
# Merge environment settings into session
123+
settings = self._session.merge_environment_settings(http_request.url, {}, None, None, None)
124+
http_response = self._session.send(http_request, **settings)
123125

124126
response = self._prepare_response(response_type, http_response)
125127
return response

src/kagglesdk/kaggle_http_client.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,9 @@ def call(
119119
self._init_session()
120120
http_request = self._prepare_request(service_name, request_name, request)
121121

122-
http_response = self._session.send(http_request)
122+
# Merge environment settings into session
123+
settings = self._session.merge_environment_settings(http_request.url, {}, None, None, None)
124+
http_response = self._session.send(http_request, **settings)
123125

124126
response = self._prepare_response(response_type, http_response)
125127
return response

0 commit comments

Comments
 (0)