Moving from 8266 to esp32 #381
-
|
Hello all,
Since it can't be possible, i'm clearly missing something and this is the question: any big topic i'm missing moving from 8266 to 32? The debug info, in case.. |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 8 replies
-
|
Hi, |
Beta Was this translation helpful? Give feedback.
-
ok wait :) not sure i got this point. Should i change my mind on any point compared with the functioning of the original asyncwebserver on ESP8266? the common point i always tried to follow is: in callbacks the very minimum of calculation (in my case it commonly creates jsons / read jsons) and if something needs to be done it is done in the loop (using vars to trigger)
It comes directly from the monitor windows in VSC+platformio. Enabled with monitor_filters = esp32_exception_decoder in platformio.ini
this is the very scary part of the message. My main.cpp includes this i hope i'm in the correct place :) and i can guarantee i did not touch any code! (yes i'm giving hope to this and to a stupid mistake to see my code come back to life 🤣 ) |
Beta Was this translation helpful? Give feedback.
-
It is possible that you have a lib that brings also another version... look inside your .pio/libdeps diectory.
you definitely need to work on a MRE I think... Is it possible otherwise that you are queuing too many websocket messages, or too big and you do not have enough heap left ? |
Beta Was this translation helpful? Give feedback.
-
I've just tried to clean the code from other async libraries, but same behavior. I'll check all libraries i have in libdeps.. thanks
yesterday trying to debug the issue i got the heap at each step, and honestly it does not seem to be the problem (the client receives only the first send, the config The sizes of the json sent: with new libraries i can no longer declare the size of jsonducuments, but in the old 8266 code all the buffers together were below 2kb, again i think this is not the issue. |
Beta Was this translation helpful? Give feedback.
-
|
Here: AsyncWebSocketMessageBuffer * buffer = ws.makeBuffer(len); // creates a buffer (len + 1) for you.
if (buffer) {
serializeJson(root, (char *)buffer->get(), len + 1);You are overflowing the array. should be: AsyncWebSocketMessageBuffer * buffer = ws.makeBuffer(len);
if (buffer) {
serializeJson(root, buffer->get(), len); |
Beta Was this translation helpful? Give feedback.
-
|
Since my MRE is as short as hell and still crashes, i was already thinking about json management, also looking at this: btw, your fix seems to work, i can't understand if it's a change in the function due to arduinojson v7 or it worked for years (and is working right now in my daikin splits) for no reason :) |
Beta Was this translation helpful? Give feedback.
-
|
makeBuffer implementation was changed over time, and a very long time ago. When changing to another org or upgrading major digit you cannot assume an immediate backward compatibility. |
Beta Was this translation helpful? Give feedback.
Here:
https://github.com/MassiPi/DaikinS21/blob/d6f7e9d6db6ef019a3612d11fc4c4639ae5893d6/src/main.cpp#L241-L243
You are overflowing the array.
should be: