Hardcoded ports in step4 and websocket connections

sermayoral / ha-samsungtv-encrypted

Samsung TV Encrypted Models (H & J Series) custom component for Home Assistant

Apache License 2.0

39 stars 24 forks source link

Hardcoded ports in step4 and websocket connections #66

Closed EPMatt closed 3 years ago

EPMatt commented 3 years ago

Hi @sermayoral,

After a user report in #21, who wasn't able to connect to its TV after the latest commit, and further reading the code in other SmartCrypto implementations, I checked that we've to revert ports for the connections performed in the connect method to the hardcoded value 8000.

I'm sorry for my previous mistake, this PR will fix it. :)

sermayoral commented 3 years ago

@EPMatt yeah, I could read a little about people problems. I'm sorry but i have no time to help you with this nowadays. A baby came...

Maybe we should create a branch to work on websocket connections. There is enought people to test it before merge in the master branch.

Thank you for your effort :-)