Fixes two more test failures. 2 left.

AArnott · AArnott · commit 61b4d2bc37a8 · 2013-03-26T06:40:37.000-07:00
diff --git a/src/DotNetOpenAuth.Core/Messaging/Channel.cs b/src/DotNetOpenAuth.Core/Messaging/Channel.cs
@@ -1181,6 +1181,10 @@ protected virtual async Task ProcessIncomingMessageAsync(IProtocolMessage messag
 					messageAccessor.ToStringDeferred());
 			}
 
+			if (this.IncomingMessageFilter != null) {
+				this.IncomingMessageFilter(message);
+			}
+
 			MessageProtections appliedProtection = MessageProtections.None;
 			foreach (IChannelBindingElement bindingElement in this.IncomingBindingElements) {
 				Assumes.True(bindingElement.Channel != null); // CC bug: this.IncomingBindingElements ensures this... why must we assume it here?
@@ -1231,10 +1235,6 @@ protected virtual async Task ProcessIncomingMessageAsync(IProtocolMessage messag
 			// message deserializer did for us.  It would be too late to do it here since
 			// they might look initialized by the time we have an IProtocolMessage instance.
 			message.EnsureValidMessage();
-
-			if (this.IncomingMessageFilter != null) {
-				this.IncomingMessageFilter(message);
-			}
 		}
 
 		/// <summary>
diff --git a/src/DotNetOpenAuth.Test/OpenId/AuthenticationTests.cs b/src/DotNetOpenAuth.Test/OpenId/AuthenticationTests.cs
@@ -270,7 +270,7 @@ private async Task ParameterizedAuthenticationTestAsync(Protocol protocol, bool
 					}
 				}
 
-				var assertionMessage = new HttpRequestMessage(HttpMethod.Get, redirectResponse.AbsoluteUri);
+				var assertionMessage = new HttpRequestMessage(HttpMethod.Get, redirectResponse);
 				if (positive) {
 					if (tamper) {
 						try {

Original file line number	Diff line number	Diff line change
`@@ -270,7 +270,7 @@ private async Task ParameterizedAuthenticationTestAsync(Protocol protocol, bool`
`270`	`270`	`}`
`271`	`271`	`}`
`272`	`272`
`273`		`- var assertionMessage = new HttpRequestMessage(HttpMethod.Get, redirectResponse.AbsoluteUri);`
	`273`	`+ var assertionMessage = new HttpRequestMessage(HttpMethod.Get, redirectResponse);`
`274`	`274`	`if (positive) {`
`275`	`275`	`if (tamper) {`
`276`	`276`	`try {`