You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
you should not add new pure virtual function to Arduino API base classes. other networking libraries don't implement them. now Ethernet, UIPEthernet and other Arduino networking libraries can't be used with this core. esp8266 did it in 2.5.0 and now they fix it for 2.5.1 and now esp32 core developers do the same mistake?
Any update or workaround for this guys? it breaks MqttClient implementation (and other libs that depends on wifiClient i guess) MqttClient mqttClient(wifiClient);
Currently we can use the older version (v3.0.7) to make it work, but i think this is not a good long term solution as we will miss all latest updates
6 commit comments
JAndrassy commentedon Apr 24, 2019
you should not add new pure virtual function to Arduino API base classes. other networking libraries don't implement them. now Ethernet, UIPEthernet and other Arduino networking libraries can't be used with this core. esp8266 did it in 2.5.0 and now they fix it for 2.5.1 and now esp32 core developers do the same mistake?
beegee-tokyo commentedon Apr 24, 2019
Same problem with TinyGSM after this commit. Doesn't compile anymore.
me-no-dev commentedon Apr 25, 2019
so with or without this change some things do not work. Let's find a common ground and make it work for all. Any proposals?
JAndrassy commentedon Apr 25, 2019
see how the esp8266 core handled it
or like Erhernet library has it would be even better https://www.arduino.cc/en/Reference/EthernetClientSetConnectionTimeout
JAndrassy commentedon May 14, 2019
you still didn't fix this? why do you need to have the
connect
version with timeout in the base class?Ibrahimsyah commentedon Dec 22, 2024
Any update or workaround for this guys? it breaks MqttClient implementation (and other libs that depends on wifiClient i guess)
MqttClient mqttClient(wifiClient);
Currently we can use the older version (v3.0.7) to make it work, but i think this is not a good long term solution as we will miss all latest updates