-
Notifications
You must be signed in to change notification settings - Fork 62
Closed
Labels
BugSomething isn't workingSomething isn't working
Description
This way customers have access to correlation ids for all ServiceExceptions caused by the service.
msgraph-sdk-dotnet-core/src/Microsoft.Graph.Core/Requests/Middleware/RedirectHandler.cs
Line 65 in f11798e
| Message = ErrorConstants.Messages.LocationHeaderNotSetOnRedirect, |
In this example, ServiceException thrown when there is no Location header for a redirect is an issue as we don't capture the requestId from the response headers.
Expected
throw new ServiceException(
new Error
{
Code = ErrorConstants.Codes.GeneralException,
Message = ErrorConstants.Messages.LocationHeaderNotSetOnRedirect,
},
response.ResponseHeaders,
response.StatusCode
);
Actual
throw new ServiceException(
new Error
{
Code = ErrorConstants.Codes.GeneralException,
Message = ErrorConstants.Messages.LocationHeaderNotSetOnRedirect,
});
Metadata
Metadata
Assignees
Labels
BugSomething isn't workingSomething isn't working