struct lws_esp32 too small to hold long SSID or Password about lws-esp32-factory

[lws-team]

OK... in lws I cranked esp32.ssid / password / active_ssid to all be [64], along with a temp array used in the scan protocol. I updated -factory and the -test-server-demos to use the new lws.

Please give it a try.

[FredrikFornstad]

Now connecting to my AP as it should. It also connects after reboot as well without going cyclic needing a erase_flash as it did before.

Not sure if this fix introduced other problems, but the lws-esp32-factory is not as robust as I would have believed. I will try to enter my findings in separate issues so you can deal with them one by one if/when you have time. Please let me know if I start to become a pain and then I will back off a bit... :-)

Overall, I find the whole lws-esp32 app REALLY nice, and if it works as I think it does, it will be perfect for my Little Project of connecting/monitoring a machine with a few sensors (using an odd serial interface) and a few switches. But your code is extensive and I must admit that I get lost in it. It can be me who is not an expert in programming, but some additional high level comments here and there would be of much help. Also, it could maybe have a bit more modular structure (or maybe it is just me who has not understood the structure yet...).

[lws-team]

Not sure if this fix introduced other problems

Doubt it. esp-idf changing all the time "introduces other problems".

lws and this ESP32 stuff is FOSS... I don't have any relationship to Espressif and nobody pays me to look after it or be "tech support". So I would hope to get fixes contributed back.

it could maybe have a bit more modular structure

If you feel that's stopping you sending patches, send patches for that :-)