[GH-ISSUE #713] Security help wanted #596

Closed
opened 2026-02-28 01:26:09 +03:00 by kerem · 2 comments
Owner

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

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
kerem 2026-02-28 01:26:09 +03:00
Author
Owner

@tablatronix commented on GitHub (Aug 29, 2018):

change GET to POST in the form see development branch

<!-- gh-comment-id:417065117 --> @tablatronix commented on GitHub (Aug 29, 2018): change GET to POST in the form see development branch
Author
Owner

@dmalham commented on GitHub (Aug 30, 2018):

Thanks! That does the trick

Dave
<!-- gh-comment-id:417256988 --> @dmalham commented on GitHub (Aug 30, 2018): Thanks! That does the trick Dave
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/WiFiManager#596
No description provided.