WIP: Added OTA updater option from esp8266

[sidey79]

Added OTA update support in config portal page

[sidey79]

@tablatronix
I Would like to know, if this is a feature you are willing to merge, before i put more work on it.
Otherwise, i can also provide an additional class which is a parent from your class.

[tablatronix]

I doubt it needs to be in core. Unless its very lightweight

[tablatronix]

I thought about refactoring menu and routes, so you can easily add menuoptions and routes, with callouts then you can just add this stuff fairly easily without modifying the lib.

[sidey79]

The libs needed seems to be already included per default in esp8266 core.

But there are some additional strings needed for the pages. Not sure if this is lightweight enough.

If you provide callbacks i could separate the update stuff in another class. But it would need some access to the server object.

[tablatronix]

#461

[sidey79]

In general, the pull request works on eps8266. But i haven't tested it on an ESP32.

Im also not sure, what breaks the example tests.

[liebman]

This is the error:

In file included from /home/travis/build/tzapu/WiFiManager/examples/AutoConnectWithStaticIP/AutoConnectWithStaticIP.ino:2:0:
/usr/local/share/arduino/libraries/WiFiManager/WiFiManager.h:46:25: fatal error: Updater.h: No such file or directory

Maybe that does not exist for the esp32 or has a different name?

[sidey79]

I can't believe it...

https://github.com/espressif/arduino-esp32/tree/master/libraries/Update/src

Update.h Updater.cpp

[liebman]

You should install the esp32 Arduino and test compile as there are some class differences.....

/usr/local/share/arduino/libraries/WiFiManager/WiFiManager.cpp: In member function 'void WiFiManager::handleUpdating()':
/usr/local/share/arduino/libraries/WiFiManager/WiFiManager.cpp:2419:3: error: 'stopAll' is not a member of 'WiFiUDP'
   WiFiUDP::stopAll();
   ^
/usr/local/share/arduino/libraries/WiFiManager/WiFiManager.cpp:2421:34: error: 'class EspClass' has no member named 'getFreeSketchSpace'
   uint32_t maxSketchSpace = (ESP.getFreeSketchSpace() - 0x1000) & 0xFFFFF000;
                                  ^

[tablatronix]

I would file that as a bug with esp32 thats messed up, and yeah ota for esp32 is fairly different, Although converging the 2 is probably on the roadmap

[sidey79]

I corrected the compile errors with esp32.

I haven't tested the ota feature on an ESP32. But on 8266 it works

[tablatronix]

Merged into ota branch for testing.
i had to fix your urls they were backwards and didnt work.

I an probably going to pull this out into an example with user code insread of core, but this is working and testable, I added todos to code

[tablatronix]

you need to fix your strings menu_ids it is missing the update token _nummenutokens index must match HTTP_PORTAL_MENU index

[tablatronix]

3e2f8ae

[sidey79]

sorry, i forgot to check this

[sidey79]

I'm not sure what ist open to geht this also in the Development branch.

One thing which ist absolutely a must ist a password protection i think.

[tablatronix]

There is an ota branch for this, it works

[sidey79]

I know. I'm testing the ota branch currently in my project.
But what ist your plan? Die you want to maintain this as a seperate Version or merge it into the development branch?

[tablatronix]

it has not been tested well and probably needs some minor improvements