-
Notifications
You must be signed in to change notification settings - Fork 565
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
Web Sockets Streamed over Https, multiple errors #470
Comments
Looks like PR #472 is addressing the HttpChannelFactory part of this. Thanks @StephenBonikowsky . |
* Opened Issue dotnet#468 for test case WebSocketHttpRequestReplyBinaryStreamed and WebSocketHttpDuplexBinaryStreamed. * Opened Issue dotnet#470 for test case public static void WebSocketHttpsDuplexBinaryStreamed. * These tests run and pass on desktop using the Bridge. * These tests are a partial fix of issue 384 * Placing these tests in a new folder called "Extensibility" based on the logic used for locating previously released WCF Samples. * Opened issue 425 to move other WebSocket tests to this location * The new Contracts include operations not used by current set of tests, leaving as is because forthcoming tests will use these operations. * Removed and sorted using statements wherever I saw that it needed it. * Incorporated feedback from initial commit
The current problem is ClientWebSocket is throwing an exception on the connect call with an error saying it was unable to connect to the remote host. |
Thanks @mconnew. What is the next step? Are you preparing a PR with a fix? |
Secure websockets streaming tests are failing because of a bug in WebSockets. See dotnet/corefx#4429 |
These tests are passing. |
Reopening this issue because this test does not pass for me when I run locally self-hosted. I set PSEXEC_PATH to the folder for psexec.exe, set IncludeTestsWithIssues=true, and run all OuterLoops. Note that Issue(470) is still applied to this test so it is not being run by CI or Helix. It is possible a later version of the packages we're using may fix this. I was running with beta-24322-03 for CoreFx when I saw this. If later packages do fix it please remove the Issue attribute from the test. Stack trace is:
|
I have confirmed this is a reliable reproducible instance of dotnet/corefx#4429 |
Thanks @mconnew -- it looks like https://github.com/dotnet/corefx/issues/4429 is slotted to be fixed in 1.2 |
@mconnew please check if the corefx fix is included in 1.1 branch. |
@mconnew has ported the fix made in corefx already make in 1.1. The remaining task is to re-enable the tests that are disabled by this issue. |
Seems to be working now. Closing. |
The test case...
WebSocketHttpsDuplexBinaryStreamed located...
C:\OSS\wcf\src\System.Private.ServiceModel\tests\Scenarios\Extensibility\WebSockets\WebSocketTests.cs
Is failing with two issues, the first one throws a "System.PlatformNotSupportedException : Channel shape not supported" from...
C:\OSS\wcf\src\System.Private.ServiceModel\src\System\ServiceModel\Channels\HttpsChannelFactory.cs
Line 78.
Replace with...
return (TChannel)(object)new ClientWebSocketTransportDuplexSessionChannel((HttpChannelFactory)(object)this, this.ClientWebSocketFactory, address, via, this.WebSocketBufferPool);
Next exception is...
System.ArgumentException occurred
Message=Empty string is not a valid subprotocol value. Please use "null" to specify no value.
Parameter name: subProtocol
ParamName=subProtocol
Source=System.Net.WebSockets.Client
StackTrace:
at System.Net.WebSockets.WebSocketValidate.ValidateSubprotocol(String subProtocol)
From...
C:\OSS\wcf\src\System.Private.ServiceModel\src\System\ServiceModel\Channels\CoreClrClientWebSocketFactory.cs
Line 19
The text was updated successfully, but these errors were encountered: