reduce reqwest timeout to 5s #557
Žádní posuzovatelé
Štítky
Bez štítků
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
Žádný milník
Žádný projekt
Žádní přiřazení uživatelé
2 účastníků
Oznámení
Termín dokončení
Žádný termín dokončení.
Závislosti
Nejsou nastaveny žádné závislosti.
Reference: Plume/Plume#557
Načítání…
Přidat tabulku
Odkázat v novém problému
K dispozici není žádný popis.
Odstranit větev „reduce-timeout“
Smazání větve je trvalé. Přestože zrušená větev může existovat i po krátkou dobu, než bude skutečně odstraněna, NELZE ji většinou vrátit. Pokračovat?
Current timeout is 30s, this add a second kind of timeout : if connection to remote can't be established in 5s, the remote is considered down, so that a thread don't stall for too long for no reason (and also make some unit test much faster to run)
It makes me think: couldn't we have a unique client (in a
lazy_static!
or something like that). I think reqwest keep connections for a certain time if the client is not dropped, and it probably caches DNS too.Otherwise, it looks good to me, so if the CI is happy you can merge.
Codecov Report
maybe not a unique client (if we have multiple request to do at the same time they will be queued), but we could probably make some kind of pool of clients. I'll keep the idea somewhere for when I have spare time to spend