tonyp7 / esp32-wifi-manager

Captive Portal for ESP32 that can connect to a saved wireless network or start an access point where you can connect to existing wifis.
MIT License
664 stars 217 forks source link

Size +1 for files served via https #32

Closed liber-tas closed 5 years ago

liber-tas commented 5 years ago

When http_server.c copies the javascript / jquery / json / html files, it calculates the length incorrectly which ends up copying one byte past the end of the text. This breaks the javascript.

0001-Subtracted-one-from-all-file-size-calculations.patch.txt

tonyp7 commented 5 years ago

Hello, I am not seeing the problem. If i request 192.168.1.1/jquery.js for instance I can see the JS has the correct end. Same for the other files

tonyp7 commented 5 years ago

I've tried this patch, it truncates all the pages by a character.

For instance becomes </html