-
Notifications
You must be signed in to change notification settings - Fork 44
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
Support PollingOverWebSocket for net60 Services #259
base: main
Are you sure you want to change the base?
Conversation
07eccfc
to
0b9d243
Compare
be131c3
to
39d70c2
Compare
607513e
to
1fa23ab
Compare
@@ -8,7 +8,7 @@ static class WebSocketSslCertificateHelper | |||
{ | |||
internal static void AddSslCertToLocalStore() | |||
{ | |||
var store = new X509Store(StoreName.My, StoreLocation.LocalMachine); | |||
var store = new X509Store(StoreName.My, StoreLocation.CurrentUser); |
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.
Linux LocalMachine is read-only
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 change is adding support for websockets on linux?
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.
Ability to install the cert on Linux, we still have an issue where we can't run any tests due to pollingoverwebsockets Client not being able to bind to an SSL port on Linux
source/Halibut.Tests/Support/TestAttributes/FailedWebSocketTestsBecomeInconclusiveAttribute.cs
Outdated
Show resolved
Hide resolved
#if SUPPORTS_WEB_SOCKET_CERTIFICATE_VALIDATION_CALLBACK | ||
clientWebSocket.Options.RemoteCertificateValidationCallback = WebSocketServerCertificateValidationCallback(clientWebSocket.Options.RemoteCertificateValidationCallback, connectionId); | ||
#endif | ||
ServicePointManager.ServerCertificateValidationCallback = ServerCertificateValidationCallback(ServicePointManager.ServerCertificateValidationCallback); |
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.
Do we need both to be registered?
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.
I don't think so. I'll have a play and see!!
[sc-53419] |
This pull request has been linked to Shortcut Story #53419: Enable WebSocket support on net60 to increase test coverage (just not Linux). |
f1a359b
to
cb97279
Compare
Background
This PR adds support for WebSocket Services on net60. This will allow net60 Tentacles on Windows and Linux to support PollingOverWebSockets
This does not change the Client requirements e.g. Octopus Server, which must still be run on Windows (net48 or net60) as
HttpListener
doesn't support SSL binding in net60 on LinuxResults
Before
PollingOverWebSocket Services (Tentacles) had to be run on Windows under net48
After
PollingOverWebSocket Services (Tentacles) can be run on Windows under net48 and net60 and on Linux under net60
How to review this PR
Quality ✔️
Pre-requisites