Replaced API requirements for HttpRequestInfo with HttpRequestBase (new in .NET 3.5 SP1).

This makes us more friendly to MVC as well as mock-based unit testing.

1 files changed, 3 insertions, 8 deletions

diff --git a/projecttemplates/MvcRelyingParty/Controllers/AuthController.cs b/projecttemplates/MvcRelyingParty/Controllers/AuthController.cs
index 9cc6e15..446c6ac 100644
--- a/projecttemplates/MvcRelyingParty/Controllers/AuthController.cs
+++ b/projecttemplates/MvcRelyingParty/Controllers/AuthController.cs
@@ -121,14 +121,9 @@ namespace MvcRelyingParty.Controllers {
 		public ActionResult LogOnPostAssertion(string openid_openidAuthData) {
 			IAuthenticationResponse response;
 			if (!string.IsNullOrEmpty(openid_openidAuthData)) {
-				var auth = new Uri(openid_openidAuthData);
-				var headers = new WebHeaderCollection();
-				foreach (string header in Request.Headers) {
-					headers[header] = Request.Headers[header];
-				}
-
 				// Always say it's a GET since the payload is all in the URL, even the large ones.
-				HttpRequestInfo clientResponseInfo = new HttpRequestInfo("GET", auth, auth.PathAndQuery, headers, null);
+				var auth = new Uri(openid_openidAuthData);
+				HttpRequestBase clientResponseInfo = new HttpRequestInfo("GET", auth, headers: Request.Headers);
 				response = this.RelyingParty.GetResponse(clientResponseInfo);
 			} else {
 				response = this.RelyingParty.GetResponse();
@@ -170,7 +165,7 @@ namespace MvcRelyingParty.Controllers {
 				}
 
 				// Always say it's a GET since the payload is all in the URL, even the large ones.
-				HttpRequestInfo clientResponseInfo = new HttpRequestInfo("GET", auth, auth.PathAndQuery, headers, null);
+				HttpRequestBase clientResponseInfo = new HttpRequestInfo("GET", auth, headers: headers);
 				response = this.RelyingParty.GetResponse(clientResponseInfo);
 			} else {
 				response = this.RelyingParty.GetResponse();