New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[System.ServiceModel] Fix timeout defaulting to 0 seconds #1656
Conversation
@@ -149,7 +160,7 @@ internal MetadataSet GetMetadataInternal (EndpointAddress address, MetadataExcha | |||
var asClientChannel = proxy as IClientChannel; | |||
if (asClientChannel == null) | |||
throw new InvalidOperationException ("The channel factory must return an IClientChannel implementation"); | |||
asClientChannel.OperationTimeout = OperationTimeout; | |||
asClientChannel.OperationTimeout = this.OperationTimeout; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why this change?
510e64a
to
8d88614
Compare
build |
1 similar comment
build |
Well, I'm no expert on WCF, but this looks sensible to me. @migueldeicaza what do you think? |
build |
I'm seeing clean on x86/amd64 and on arm I see
which is an intermittent failure that I've been seeing on most PRs on arm. |
@@ -322,8 +321,19 @@ public void Dispose () | |||
} | |||
} | |||
|
|||
[MonoTODO] | |||
public TimeSpan OperationTimeout { get; set; } | |||
TimeSpan? operation_timeout = null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should go to the top with all the other fields.
LGTM |
8d88614
to
e4d491c
Compare
Moved it up. |
[System.ServiceModel] Fix timeout defaulting to 0 seconds
[System.ServiceModel] Fix timeout defaulting to 0 seconds Commit migrated from mono/mono@1f8da92
No description provided.