-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add support for WSS / SSL #65
Comments
Ideally, we would get hold of the root CA and trust that. Unfortunately, the TV doesn't present it, only the intermediate, although we could trust that while allowing a partial chain if we cannot obtain the root CA.
|
The changes were easy enough to make and work for me. You called it a proof of concept, but this seems fine, apart from maybe verifying the certificate as I suggested. I don't think this needs to be configurable. I'd be surprised if any of the models already supported by this library don't support SSL. What do you think? |
I got the partial chain verification to work. I had hoped to fish the root CA out of the Android app, but I cannot find it, despite finding a bunch of other certificates. I'll try emailing [email protected]. It's worth a shot, right? I can submit this in a pull request, but would you merge it, @bendavid? I know you haven't touched this for ages, but I don't want to have to maintain yet another fork. |
It's already done in bscpylgtv. |
I updated my TV firmware and now it is rejecting non-SSL WS connection. The API still works with WSS with SSL. However, aiopylgtv doesn't currently support that.
To support SSL we need the following changes:
3001
(from3000
)wss://
instead ofws://
As a proof of concept, I got it working by changing the following code:
WebOsClient.connect_handler
:The text was updated successfully, but these errors were encountered: