search

atxbyea / samsungrac

Home Assistant Climate Device for controlling (not only) Samsung AC
60 stars 18 forks source link

TLSv1 explicitly needed for TP6X_RAC_16K #101

Open BZoli97 opened 2 months ago

BZoli97 commented 2 months ago

Hello,

I have an AC "TP6X_RAC_16K" and from HA 2024.07 it is stops to working because still using TLSv1

ERROR (MainThread) [custom_components.climate_ip.climate] Request result exception: HTTPSConnectionPool(host='xxx.xxx.xxx.xxx', port=8888): Max retries exceeded with url: /devices (Caused by SSLError(SSLError(1, '[SSL: UNSUPPORTED_PROTOCOL] unsupported protocol (_ssl.c:1000)')))

I saw the set_ciphers was already set to "ALL:@SECLEVEL=0", and seemingly worked till now with openssl3.x but from now for me it is also require minimum_version ssl.TLSVersion.TLSv1

keyluke commented 2 months ago

Same issue here, reverted back to v6.1. Seems that the changes in connection_request.py that do not specify explicitly TLSv1 break the possibility to interact with old appliances. breakingssl1@2x

BZoli97 commented 2 months ago

I just checked the blame and missed this line was alreday there before. I just added a similar PR, but in this case that is not needed

nrpetonr commented 2 months ago

same problem here, reverted back to previous version

samuelecarpene commented 2 months ago

same problem here