fastapi icon indicating copy to clipboard operation
fastapi copied to clipboard

fix: added timeout to avoid ReadTimeout Error.

Open iudeen opened this issue 3 years ago • 6 comments

Fixes an error that occurred in scheduled actions of FastAPI People.

iudeen avatar Aug 02 '22 15:08 iudeen

Codecov Report

Merging #5222 (3461db3) into master (0195bb5) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #5222   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          539       539           
  Lines        13902     13902           
=========================================
  Hits         13902     13902           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Aug 02 '22 15:08 codecov[bot]

📝 Docs preview for commit 5e5352fd0b146655beda89b70959302247fa2ce8 at: https://62e93f5124c2231a962ef181--fastapi.netlify.app

github-actions[bot] avatar Aug 02 '22 15:08 github-actions[bot]

Should I drive with an environment variable instead of hard-coding 30?

iudeen avatar Aug 02 '22 15:08 iudeen

📝 Docs preview for commit 76cbeb1d40e4e8a7db7da3aaee7892972d730db9 at: https://62e943bddc072d1e10747d7a--fastapi.netlify.app

github-actions[bot] avatar Aug 02 '22 15:08 github-actions[bot]

@tiangolo this might be trivial. Stumbled upon actions tab and found that this action is failing for last 2 months, and this might fix it.

Also there might be a need to configure a env variable (optional, defaults to 30).

iudeen avatar Aug 03 '22 21:08 iudeen

📝 Docs preview for commit 9487385249775eaca124a088356659726c6c8056 at: https://62f0e65fac35f861ecd93ca7--fastapi.netlify.app

github-actions[bot] avatar Aug 08 '22 10:08 github-actions[bot]

📝 Docs preview for commit c6735f8c9921178f22e137d88b8db7f56bc78908 at: https://62fdbbf2362b7b55454f112a--fastapi.netlify.app

github-actions[bot] avatar Aug 18 '22 04:08 github-actions[bot]

📝 Docs preview for commit 45cab92331fa8f89b10b3b1a09093744ca876d50 at: https://62fec0c48802d000a68820fe--fastapi.netlify.app

github-actions[bot] avatar Aug 18 '22 22:08 github-actions[bot]

📝 Docs preview for commit feb10df210e9e23d437fb3c7903f7218fc1aa1ce at: https://6305e6e06d6fbc5a5e9b9fad--fastapi.netlify.app

github-actions[bot] avatar Aug 24 '22 08:08 github-actions[bot]

📝 Docs preview for commit 8e8fe07815236dfad5a3b27a49c9cc217122a944 at: https://6308a074cf308e78b9e75037--fastapi.netlify.app

github-actions[bot] avatar Aug 26 '22 10:08 github-actions[bot]

📝 Docs preview for commit 45c172b5fec01c488549dd1b6a38212154a7f023 at: https://6308d96970db72283dcc5e59--fastapi.netlify.app

github-actions[bot] avatar Aug 26 '22 14:08 github-actions[bot]

📝 Docs preview for commit e75dff047817be5ec9a30feb679887fdc7ae86b0 at: https://6308e5cdfd6621375f0653de--fastapi.netlify.app

github-actions[bot] avatar Aug 26 '22 15:08 github-actions[bot]

📝 Docs preview for commit 4d7bfbb0db7b035eb5591c63eecb5ab4f2faadc4 at: https://6312e999e31c9631267850f8--fastapi.netlify.app

github-actions[bot] avatar Sep 03 '22 05:09 github-actions[bot]

📝 Docs preview for commit 1ab76a1ae8b7ca0d1e4df07f4156e886c516256a at: https://6312f04e7f1ade5a2d6d8224--fastapi.netlify.app

github-actions[bot] avatar Sep 03 '22 06:09 github-actions[bot]

@tiangolo is this something you will be interested in? If not, we can close :)

iudeen avatar Sep 04 '22 09:09 iudeen

📝 Docs preview for commit 39f5fbbb483ab66693645e7ec471366507fbf784 at: https://63146c8b00ed394bc6d435d5--fastapi.netlify.app

github-actions[bot] avatar Sep 04 '22 09:09 github-actions[bot]

📝 Docs preview for commit 92a1eabdf490359b7595ae6a0c4b97655380096d at: https://6314bb0567c58a7ae2aa7ce3--fastapi.netlify.app

github-actions[bot] avatar Sep 04 '22 14:09 github-actions[bot]

Nice, thank you @iudeen! :bow:

I updated it to simplify the code a bit. Thanks for your contribution! :cake:

tiangolo avatar Sep 04 '22 15:09 tiangolo