Merge pull request #41 from notion-dotnet/bfix/40-errorResponse-is-null-on-at-least-one-execution-path

KoditkarVedant · web-flow · commit 605889f988f7 · 2021-07-17T15:22:18.000+05:30
Fix 'errorResponse' is null on at least one execution path. 🐛
diff --git a/Src/Notion.Client/RestClient/RestClient.cs b/Src/Notion.Client/RestClient/RestClient.cs
@@ -69,7 +69,7 @@ private static async Task<Exception> BuildException(HttpResponseMessage response
                 }
             }
 
-            return new NotionApiException(response.StatusCode, errorResponse?.ErrorCode, errorResponse.Message);
+            return new NotionApiException(response.StatusCode, errorResponse?.ErrorCode, errorResponse?.Message);
         }
 
         private async Task<HttpResponseMessage> SendAsync(

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ private static async Task<Exception> BuildException(HttpResponseMessage response`
`69`	`69`	`}`
`70`	`70`	`}`
`71`	`71`
`72`		`- return new NotionApiException(response.StatusCode, errorResponse?.ErrorCode, errorResponse.Message);`
	`72`	`+ return new NotionApiException(response.StatusCode, errorResponse?.ErrorCode, errorResponse?.Message);`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`private async Task<HttpResponseMessage> SendAsync(`