Add request timeouts to federation HTTP calls
@debdutdeb ci sta già lavorando.
Dal 16/9/2025.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Description
Federation HTTP requests in FederationRequestService.makeSignedRequest() currently lack timeout handling, which can cause requests to hang indefinitely on network issues.
Context
The @hs/core fetch implementation doesn't include built-in timeout support. This was identified during review of PR #191.
Proposed Solution
Add configurable timeout support using one of these approaches:
- Use AbortController with setTimeout to cancel long-running requests
- Extend
@hs/corefetch to accepttimeoutMsorsignalparameters - Add a configurable timeout (e.g.,
FEDERATION_REQUEST_TIMEOUT_MS) with reasonable default (~15s)
Files Affected
packages/federation-sdk/src/services/federation-request.service.ts- Potentially
@hs/corefetch implementation
References
- Original PR: https://github.com/RocketChat/homeserver/pull/191
- Comment: https://github.com/RocketChat/homeserver/pull/191#discussion_r2351945893
/cc @debdutdeb
- Lingua principale
- TypeScript
- Stelle
- 17
- Fork
- 18
- Merge medio
- 2h 11m
- PR unite (30g)
- 2
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di RocketChat/homeserver
-
pruneEventDict and signEvent's `prune` flag should go away entirely, replaced byForse di nuovo libera @sampaiodiego l’ha presa 76 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad: type: feature
RocketChat/homeserver#411 · 1 commento · 1 assegnatario ·
-
Include banned and invited servers in event destination setForse già presa @ggazzo l’ha presa 233 giorni fa. Apertatype: bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
RocketChat/homeserver#388 ·
-
we need a map of rooms and destinations to avoid having to get rooms state just to send an event to all servers in the room.Forse di nuovo libera @sampaiodiego l’ha presa 249 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad: type: feature
RocketChat/homeserver#342 · 4 commenti · 1 assegnatario ·
-
this may not be the best place to do this validationForse di nuovo libera @sampaiodiego l’ha presa 326 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad:
RocketChat/homeserver#322 · 1 commento · 1 reazione · 1 assegnatario ·
-
check if all sendEventToAllServersInRoom should be followed by an emitterForse di nuovo libera @ggazzo l’ha presa 359 giorni fa e non c’è nessuna pull request aperta. Apertatodo :spiral_notepad:
RocketChat/homeserver#304 · 1 assegnatario ·
Tutte le issue di RocketChat/homeserver
Issue simili
-
Bug: Broken RTL / BiDi text rendering when mixing Persian (Farsi) with English or numbersForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug
Difficoltà 2/5 Mezza giornata Idoneità per principianti 68/100
amosblomqvist/learnharness#758 ·
I maintainer di solito rispondono entro 2 giorni
-
Change WorkspacesAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
bitfocus/companion-module-renewedvision-propresenter-api#134 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
rcpch/digital-growth-charts-react-component-library#304 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
dsh-market/dsh-market#1210 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
trustjonathan/STUDY-HUB#13 ·