Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wireless AP SSIDS/Passwords containing spaces fail in station/installer #90

Open
rksh opened this issue Oct 7, 2020 · 5 comments
Open

Comments

@rksh
Copy link

rksh commented Oct 7, 2020

When attempting to re-configure my homebrew setup to a home wireless AP with an SSID and Password containing multiple space (0x20) characters.

It may worth checking other corner cases such as %2F, %3F, %21, and even possible unicode.

@rksh
Copy link
Author

rksh commented Oct 7, 2020

proposed fix - works in testing, but probably not the most elegant way to do this. Provided as a proof-of-concept since I can't easily write a meaningful test case for this, and testing corner cases requires standing up and reconfiguring an 802.11 AP.

@ruffruffruffie
Copy link
Collaborator

Hi there! I'm trying to replicate this issue and I am unable to do so; I've tried with SSIDs & password combinations with roughly 5 spaces. Do you have any additional pointers that I can test this bug with?

@rksh
Copy link
Author

rksh commented Oct 7, 2020

oh no, this is going to end up being hardware/implementation specific, isn't it? Will dust off some old wireless hardware and do detailed testing.

setup that's known-bad:
Asus AC1200G WPA2 PSK, latest mfgr. firmware, 2 spaces in the SSID, 4 spaces in the password.
Micropython: esp32-idf3-20200902-v1.13.bin

@rksh
Copy link
Author

rksh commented Oct 22, 2020

also worth noting that i'm using zsh under OSX.

@rksh
Copy link
Author

rksh commented Oct 29, 2020

OK, finally stood up an old Apple Airport for testing!
SSID: "Antenny Wi-Fi Network"

With the password "Testing^5" - All good.
With the password "test test test" - no dice.

Worth noting that this is an old-as-dirt AP, but Apple has apparently updated it to support WPA2-PSK-CCMP (as registered in Android using WiGLE WiFi Wardriving) - haven't found a way to downshift it to WPA1 or WEP, they're aggressive about forcing pretty good defaults in their Airport Utility suite.

Regardless spaces in the SSID are clearly not the issue (weird!). OK to update the issue title?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants