Skip to content
This repository was archived by the owner on Mar 20, 2019. It is now read-only.

Commit 982dc39

Browse files
committed
Fixes ContentType of OAuth 1 direct responses.
22 test failures remaining.
1 parent e0a8bb0 commit 982dc39

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/DotNetOpenAuth.OAuth/OAuth/ChannelElements/OAuthChannel.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -220,10 +220,9 @@ protected override HttpRequestMessage CreateHttpRequest(IDirectedProtocolMessage
220220
protected override HttpResponseMessage PrepareDirectResponse(IProtocolMessage response) {
221221
var messageAccessor = this.MessageDescriptions.GetAccessor(response);
222222
var fields = messageAccessor.Serialize();
223-
string responseBody = MessagingUtilities.CreateQueryString(fields);
224223

225-
HttpResponseMessage encodedResponse = new HttpResponseMessage {
226-
Content = new StringContent(responseBody),
224+
var encodedResponse = new HttpResponseMessage {
225+
Content = new FormUrlEncodedContent(fields),
227226
};
228227

229228
ApplyMessageTemplate(response, encodedResponse);

src/DotNetOpenAuth.Test/OAuth/ChannelElements/OAuthChannelTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ public async Task SendDirectMessageResponse() {
121121

122122
var response = await this.channel.PrepareResponseAsync(message);
123123
Assert.AreEqual(HttpStatusCode.OK, response.StatusCode);
124-
Assert.AreEqual(Channel.HttpFormUrlEncodedContentType, response.Content.Headers.ContentType.MediaType);
124+
Assert.AreEqual(Channel.HttpFormUrlEncodedContentType.MediaType, response.Content.Headers.ContentType.MediaType);
125125

126126
NameValueCollection body = HttpUtility.ParseQueryString(await response.Content.ReadAsStringAsync());
127127
Assert.AreEqual("15", body["age"]);

0 commit comments

Comments
 (0)