Permalink
Browse files

Fixes two more test failures. 2 left.

  • Loading branch information...
1 parent 3883e15 commit 61b4d2bc37a83ee2bedd35b9e6df6809bfafef4d @AArnott AArnott committed Mar 26, 2013
@@ -1181,6 +1181,10 @@ public abstract class Channel : IDisposable {
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 @@ public abstract class Channel : IDisposable {
// 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>
@@ -270,7 +270,7 @@ public class AuthenticationTests : OpenIdTestBase {
}
}
- var assertionMessage = new HttpRequestMessage(HttpMethod.Get, redirectResponse.AbsoluteUri);
+ var assertionMessage = new HttpRequestMessage(HttpMethod.Get, redirectResponse);
if (positive) {
if (tamper) {
try {

0 comments on commit 61b4d2b

Please sign in to comment.