Merge pull request #267 from watson-developer-cloud/update-error-filter

mediumTaj · web-flow · commit 8f4a3ada26ad · 2018-07-18T15:20:11.000-05:00
refactor(error-filter): simplify check for headers and content type
diff --git a/src/IBM.WatsonDeveloperCloud/Http/Filters/ErrorFilter.cs b/src/IBM.WatsonDeveloperCloud/Http/Filters/ErrorFilter.cs
@@ -34,7 +34,7 @@ public void OnResponse(IResponse response, HttpResponseMessage responseMessage)
 
                 var error = responseMessage.Content.ReadAsStringAsync().Result;
 
-                if (responseMessage.Content.Headers.ContentType != null && responseMessage.Content.Headers.ContentType.MediaType == HttpMediaType.APPLICATION_JSON)
+                if (responseMessage.Content.Headers?.ContentType?.MediaType == HttpMediaType.APPLICATION_JSON)
                 {
                     exception.Error = JsonConvert.DeserializeObject<Error>(error);
                 }

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public void OnResponse(IResponse response, HttpResponseMessage responseMessage)`
`34`	`34`
`35`	`35`	`var error = responseMessage.Content.ReadAsStringAsync().Result;`
`36`	`36`
`37`		`- if (responseMessage.Content.Headers.ContentType != null && responseMessage.Content.Headers.ContentType.MediaType == HttpMediaType.APPLICATION_JSON)`
	`37`	`+ if (responseMessage.Content.Headers?.ContentType?.MediaType == HttpMediaType.APPLICATION_JSON)`
`38`	`38`	`{`
`39`	`39`	`exception.Error = JsonConvert.DeserializeObject<Error>(error);`
`40`	`40`	`}`