Commit 55e27a6

authored

Merge pull request #316 from notion-dotnet/fix-lgtm-alerts

fix lgtm alerts

2 parents cf1e4ee + 260c546 commit 55e27a6Copy full SHA for 55e27a6

File tree

-2

lines changed

-2

lines changed

Lines changed: 3 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -64,8 +64,9 @@ private static async Task<Exception> BuildException(HttpResponseMessage response`
`64`	`64`	`{`
`65`	`65`	`errorResponse = JsonConvert.DeserializeObject<NotionApiErrorResponse>(errorBody);`
`66`	`66`	`}`
`67`		`- catch`
	`67`	`+ catch(Exceptionex)`
`68`	`68`	`{`
	`69`	`+ Log.Error(ex, "Error when parsing the notion api response.");`
`69`	`70`	`}`
`70`	`71`	`}`
`71`	`72`
`@@ -84,7 +85,7 @@ public async Task<HttpResponseMessage> SendAsync(`
`84`	`85`
`85`	`86`	`requestUri = AddQueryString(requestUri, queryParams);`
`86`	`87`
`87`		`- HttpRequestMessage httpRequest = new HttpRequestMessage(httpMethod, requestUri);`
	`88`	`+ usingvar httpRequest = new HttpRequestMessage(httpMethod, requestUri);`
`88`	`89`	`httpRequest.Headers.Authorization = new AuthenticationHeaderValue("Bearer", _options.AuthToken);`
`89`	`90`	`httpRequest.Headers.Add("Notion-Version", _options.NotionVersion);`
`90`	`91`

Comments

(0)