mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-30 10:35:52 +03:00
[GH-ISSUE #713] Security help wanted #596
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#596
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 @dmalham on GitHub (Aug 29, 2018).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/713
This is a nice bit of code but one thing I don’t like is that after you have saved the WiFi credentials from the AP web page. the browser address of the new page includes both the SSID and the password in plain. This, I think, is a security hole as it stays in the browser history of the device you use to connect to the AP unless, of course, you clear the history or use the private browsing option. Not very idiot proof! Can anyone reassure me or come up with a fix I can use?
Dave
@tablatronix commented on GitHub (Aug 29, 2018):
change GET to POST in the form see development branch
@dmalham commented on GitHub (Aug 30, 2018):
Thanks! That does the trick