Merge pull request #290 from WilliamsJason/WilliamsJason_minorErrorHandling

Minor fix to avoid exception in error parsing case
This commit is contained in:
David Kline 2018-06-28 17:37:32 -07:00 коммит произвёл GitHub
Родитель 06b65c173b f458f2f7ee
Коммит c373fae13d
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
1 изменённых файлов: 13 добавлений и 10 удалений

Просмотреть файл

@ -153,18 +153,21 @@ namespace Microsoft.Tools.WindowsDevicePortal
{
HttpErrorResponse errorResponse = DevicePortal.ReadJsonStream<HttpErrorResponse>(dataStream);
error.HResult = errorResponse.ErrorCode;
error.Reason = errorResponse.ErrorMessage;
// If we didn't get the Hresult and reason from these properties, try the other ones.
if (error.HResult == 0)
if (errorResponse != null)
{
error.HResult = errorResponse.Code;
}
error.HResult = errorResponse.ErrorCode;
error.Reason = errorResponse.ErrorMessage;
if (string.IsNullOrEmpty(error.Reason))
{
error.Reason = errorResponse.Reason;
// If we didn't get the Hresult and reason from these properties, try the other ones.
if (error.HResult == 0)
{
error.HResult = errorResponse.Code;
}
if (string.IsNullOrEmpty(error.Reason))
{
error.Reason = errorResponse.Reason;
}
}
}
}