Retry handler should retry on connect timeout
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- typescript
- Domaine
- api
Piste de recherche
Start in src/middleware/RetryHandler.ts at lines 176-189, referenced by the issue, and inspect the retry middleware's existing error handling. Use the reported FetchError with connect ETIMEDOUT as the case to cover, then verify that the middleware retries it and that existing retry behavior remains intact.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Feature Request
Is your feature request related to a problem? Please describe
Currently the retry handler only rethrows when the underlining request throws:
https://github.com/microsoftgraph/msgraph-sdk-javascript/blob/130f3c0e9d329498ef8444d86181255b640d5743/src/middleware/RetryHandler.ts#L176-L189
This means that a connect timeout will not be retried. This happened in production for us the other day.
{ statusCode: -1, code: "FetchError", message: "request to https://graph.microsoft.com/v1.0/me/mailboxSettings failed, reason: connect ETIMEDOUT 40.126.6.114:443", requestId: null, date: {}, body: "FetchError: request to https://graph.microsoft.com/v1.0/me/mailboxSettings failed, reason: connect ETIMEDOUT 40.126.6.114:443" }
Describe the solution you'd like
The retry middleware should catch the connection timeout and retry.
- Langage dominant
- TypeScript
- Étoiles
- 833
- Forks
- 240
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de microsoftgraph/msgraph-sdk-javascript
-
status:waiting-for-triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 62/100
-
Is anyone here? Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
microsoftgraph/msgraph-sdk-javascript#2005 · 1 réaction ·
-
Is it really correct that the Javascript version of the graph SDK doesn't offer typed requests? Ouvertestatus:waiting-for-triage
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 42/100
-
status:waiting-for-triage type:bug
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
microsoftgraph/msgraph-sdk-javascript#1999 · 2 commentaires · 1 réaction ·
-
status:waiting-for-triage type:bug
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
Toutes les issues de microsoftgraph/msgraph-sdk-javascript
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
safetrustcr/dApp-SafeTrust#426 ·
-
area:workflow bug ready-for-agent
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
fil-donadoni/tolaria#4409 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Fission-AI/OpenSpec#1960 ·
-
Add dependabot Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
corsairdev/corsair#1764 ·