search

espressif / esp-azure

SDK to connect ESP8266 and ESP32 to Microsoft Azure IoT services
176 stars 92 forks source link

Fixes for MQTT over WebSockets (CA-119) #111

Open jspngh opened 3 years ago

jspngh commented 3 years ago

I have split off the WebSockets part from PR #68 and fixed the remarks from @shahpiyushv. I've tested this with IDF v4.2.

CLAassistant commented 3 years ago

CLA assistant check
All committers have signed the CLA.

formus14 commented 3 years ago

I have tested it out, i could verify it works fine with IDF4.1.1 as well. However it fails with IDF4.1, showing the following error: E (24562) esp-tls-mbedtls: read error :-80:

jspngh commented 3 years ago

I believe there was an issue with TLS renegotiation that was fixed and backported to v4.1 and v4.0. I think it's advisable to use the latest v4.1.x tag or track the release/v4.1 branch if you want to stay on that version.

MartinTJDK commented 3 years ago

When will fix be merged to master? I see same issue regarding TLS renegotiation, where option is not supported.

jspngh commented 3 years ago

Maybe we can tag some of the folks of Espressif who are active in this repository. @igrr @shahpiyushv @lhespress @projectgus :wave: If anything still needs to be done before this can be merged, please let me know.

formus14 commented 3 years ago

@MartinTJDK @jspngh Just FYI, The PR is working fine with idf 4.1.1 , 4.2 and 4.2.1 Only idf4.1 had this TLS renegotiation issue ;)

TLS renegotiation was fixed from IDF v4.1.1 on as shown in the release notes image