Skip to content

Commit a92b58c

Browse files
committed
Identifies a couple of TODO comments as being new in 5.0 that need to be resolved.
1 parent 5704de1 commit a92b58c

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/DotNetOpenAuth.Core/Messaging/MessagingUtilities.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2062,7 +2062,7 @@ internal HttpResponseMessageActionResult(HttpResponseMessage response) {
20622062
/// </summary>
20632063
/// <param name="context">The context in which the result is executed. The context information includes the controller, HTTP content, request context, and route data.</param>
20642064
public override void ExecuteResult(ControllerContext context) {
2065-
// TODO: fix this to be asynchronous.
2065+
// TODO 5.0: fix this to be asynchronous.
20662066
this.response.SendAsync(context.HttpContext).GetAwaiter().GetResult();
20672067
}
20682068
}

src/DotNetOpenAuth.OpenId/OpenId/UntrustedWebRequestHandler.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ protected override async Task<HttpResponseMessage> SendAsync(
312312
// We don't want to blindly set all ServicePoints to not use the Expect header
313313
// as that would be a security hole allowing any visitor to a web site change
314314
// the web site's global behavior when calling that host.
315-
// TODO: verify that this still works in DNOA 5.0
315+
// TODO 5.0: verify that this still works in DNOA 5.0
316316
var servicePoint = ServicePointManager.FindServicePoint(request.RequestUri);
317317
Logger.Http.InfoFormat(
318318
"HTTP POST to {0} resulted in 417 Expectation Failed. Changing ServicePoint to not use Expect: Continue next time.",

0 commit comments

Comments
 (0)