mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-27 09:05:56 +03:00
[GH-ISSUE #655] ESP32 development branch - 2 fixes and problem accessing OnDemandAP config page #547
Labels
No labels
📶 WiFi
🕸️ HTTP
Branch
DEV Help Wanted
Discussion
Documentation
ESP32
Example
Good First Issue
Hotfix
In Progress
Incomplete
Needs Feeback
Priority
QA
Question
Task
Upstream/Dependancy
bug
duplicate
enhancement
invalid
pull-request
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/WiFiManager#547
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Mirdox on GitHub (Jul 13, 2018).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/655
WiFimanager Branch/Release: Development
Hello there,
Thanks for this project, it works perfectly on esp8266.
Now I was trying on esp32 and I'm facing several issues.
Some of them i was able to fix.
Fix 1: There is conflict with esp32 library WebServer file HTTP_Method.h which uses same variable HTTP_HEAD. Renaming it in WiFiManager.cpp and strings_en.h to HTML_HEAD solves the problem.
HTML_HEAD is btw more appropriate, because it is HTML Header, not HTTP.
Fix 2: There is a typo in WiFiManager.cpp line 1081
bool tok_I = HTTP_PARAM_temp.indexOf(FPSTR(T_i)) > 0; // lower case i
should be changed to
bool tok_I = HTTP_PARAM_temp.indexOf(FPSTR(T_I)) > 0; // upper case I
Current problem: now I'm stuck, because I cannot open config page in browser while connected to OnDemandAP. When the Wifi Credentials are missing the config page is working fine.
There is no crash and no debug messages and I've tried on more devices.
Any idea?
Thank you
@tablatronix commented on GitHub (Jul 13, 2018):
I just used it he other day, were there alot of commits to esp32?
@tablatronix commented on GitHub (Jul 13, 2018):
Ah i see webserver abstractions, I just looked for changes this morning and missed this. Will fix