mirror of
https://github.com/tzapu/WiFiManager.git
synced 2026-04-27 17:15:53 +03:00
[GH-ISSUE #30] Invalid AP password gives no warning #22
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#22
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 @paccerdk on GitHub (Dec 22, 2015).
Original GitHub issue: https://github.com/tzapu/WiFiManager/issues/30
I've spent a few hours trying to figure out why i couldn't change the default AP SSID / password.
The module would come up with the default "AutoConnectAP" and no password,
and seemed to do so regardless of what i did.
Turns out i picked a password which was too short.
Using a password less than 8 characters results in the module using previous saved AP settings.
This is probably intended behavior, but i suggest printing a warning when using passwords <8 characters, or maybe a comment mentioning this.
@tzapu commented on GitHub (Dec 23, 2015):
hi, already mentioned here https://github.com/tzapu/WiFiManager#password-protect-the-configuration-access-point
i did not add a warning as i did not know what the criteria was, maybe simply 8 chars or longer is enough as a check
@paccerdk commented on GitHub (Dec 23, 2015):
After a bit of digging around, I've come to the following conclusions:
Pass-phrase must consist of 8 to 63 ASCII-encoded characters in the range of 32 to 126 (decimal)
A valid SSID consists of 0-32 octets with arbitrary contents.
This is according to the IEEE 802.11 standard
Source:
http://standards.ieee.org/getieee802/download/802.11-2012.pdf
@tzapu commented on GitHub (Dec 23, 2015):
thanks for this
added warning in latest commit