microsoftgraph / microsoftgraph/msgraph-sdk-java
Improve default RetryHandler implementation
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 444
- Forks
- 154
- Ø Merge
- 18 Std. 28 Min.
- Gemergte PRs (30 T.)
- 4
Beschreibung
Is your feature request related to a problem? Please describe the problem.
SDK V5
As per incident: 2409111420000033
in case there is a low memory on server side the server returns 503 for other error codes.
In such cases SDK treats 404, 403 errors as if it is 503 and retries.
We were instructed by Microsoft support person of contact to implement RetryHandler on application side and retry only if "retry-after" header is present. However, it sounds like this should be handled in SDK side rather than us.
Describe the solution you'd like.
I would propose such cases, when Microsoft servers have low memory to be handled by SDK
Additional context?
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Das Issue nennt keine Quelldatei und keinen Test; beginne damit, die Standardimplementierung von RetryHandler und die vorhandenen Retry-Tests im Java SDK zu finden. Prüfe, wie 403, 404, 503 und der retry-after header behandelt werden, und füge anschließend Tests hinzu, die zeigen, dass Retries nur ausgeführt werden, wenn der angeforderte Response-Header sie erlaubt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100