Skip to content

Commit 72bcab6

Browse files
Add TODO to review whether Accept is redundant
Add a TODO comment to review whether adding the Accept header is redundant.
1 parent 7313320 commit 72bcab6

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/AspNet.Security.OAuth.VisualStudio/VisualStudioAuthenticationHandler.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,10 @@ protected override async Task<OAuthTokenResponse> ExchangeCodeAsync([NotNull] OA
7171
}
7272

7373
using var request = new HttpRequestMessage(HttpMethod.Post, Options.TokenEndpoint);
74+
75+
// TODO Review whether this is just a copy-paste mistake and is redundant by testing with a real instance
7476
request.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue("application/x-www-form-urlencoded"));
77+
7578
request.Content = new FormUrlEncodedContent(tokenRequestParameters);
7679

7780
using var response = await Backchannel.SendAsync(request, Context.RequestAborted);

0 commit comments

Comments
 (0)