Add timeout to UserSecretsClient · messerzen/docker-python@fe26648 · GitHub

messerzen / docker-python Public

forked from Kaggle/docker-python

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit fe26648

Browse filesBrowse the repository at this point in the historyBrowse files

vimota

committed

Add timeout to UserSecretsClient

1 parent a16bc4b commit fe26648
Copy full SHA for fe26648

1 file changed

+2-1Lines changed: 2 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• patches
  • kaggle_secrets.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎patches/kaggle_secrets.py‎

Copy file name to clipboardExpand all lines: patches/kaggle_secrets.py

+2-1Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -11,6 +11,7 @@
11	11	_KAGGLE_DEFAULT_URL_BASE = "https://www.kaggle.com"
12	12	_KAGGLE_URL_BASE_ENV_VAR_NAME = "KAGGLE_URL_BASE"
13	13	_KAGGLE_USER_SECRETS_TOKEN_ENV_VAR_NAME = "KAGGLE_USER_SECRETS_TOKEN"
	14	+ TIMEOUT_SECS = 10
14	15
15	16
16	17	class CredentialError(Exception):
@@ -43,7 +44,7 @@ def _make_post_request(self, data):
43	44	req = urllib.request.Request(url, headers=self.headers, data=bytes(
44	45	json.dumps(request_body), encoding="utf-8"))
45	46	try:
46		- with urllib.request.urlopen(req) as response:
	47	+ with urllib.request.urlopen(req, timeout=TIMEOUT_SECS) as response:
47	48	response_json = json.loads(response.read())
48	49	if not response_json.get('wasSuccessful') or 'result' not in response_json:
49	50	raise BackendError(

0 commit comments

Comments

0 (0)