microsoftgraph / microsoftgraph/msgraph-sdk-java
Body missing in ApiException
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.
In v6 I am missing a method to get the body of a failed request, which may help to reproduce an issue.
With v5 I'm able to call graphServiceException.getMessage(true); which contains the headers and body of the failed request. However in v6 there is a method for the headers apiException.getResponseHeaders(), but none for the body.
Question on that: Are these really the responseHeaders and not the requestHeaders of the failed request? I don't see why someone would need the headers of the response message. So maybe this case is just a naming issue.
Describe the solution you'd like.
Add method getRequestBody or getResponseBody analogue to getRequestHeaders or getResponseHeaders
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
Beginnen Sie mit ApiException und vergleichen Sie den vorhandenen getResponseHeaders()-Accessor mit dem im Issue beschriebenen Verhalten von v5 GraphServiceException.getMessage(true). Ermitteln Sie, ob der Body der fehlgeschlagenen Antwort beibehalten wird und ob es sich bei den vorhandenen Headern um Antwort- oder Anfrage-Header handelt. Als erledigt gilt die Aufgabe, wenn der vorgesehene Body-Accessor und die Benennung der Header geklärt und durch die relevanten Tests des Projekts abgedeckt sind.
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
- 38/100