-
Notifications
You must be signed in to change notification settings - Fork 9
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
Cannot add wifi credentials in the GUI. #55
Comments
Log no wifi: 12:49:32][ 13394][E][WiFiUdp.cpp:185] endPacket(): could not send data: 118 |
Log Wifi OK: [12:50:51]Not-Found HTTP call: URI: /favicon.ico |
Additional feedback from Monsoono: it's possible the device wasn't able to handle incorrectly entered wifi credentials. If this is the case, perhaps it needs to fallback to broadcasting its own hotspot (not sure if it does this already). |
Hey @ewowi, still unable to connect WiFi. I deleted the StarLeds repository and reinstalled a clean version and I'm still running into the issue. The gui loads extremely slow for me as well on desktop compared to phone browser. Desktop version cannot submit wifi credentials most of the time. If it does or I used phone to submit it crashes immediately. The logs are below. Probably including a bit more than needed.
|
A different attempt. GUI loaded fast this time on desktop and submit button worked.
|
@Brandon502 Regarding: #1 0x400e57f9:0x3ffdbb40 in SysModNetwork::initConnection() at src/Sys/SysModNetwork.cpp:128 (discriminator 3) This is the related code:
So something wrong with strlen(password), I can only imagine that goes wrong if password is nullptr, but that shouldn't be the case. I am preparing a commit where password not nullptr will be checked. Will let you know when available |
Regarding: Connecting to WiFi DNet / ******************************9 Looks like a password of 43 bytes while code expects no longer then 31/32 (that 9 at position 37 is a bit strange though) The code is supposed to limit the max to 32: How long was the password you entered? (In next commit I will limit the string length of said and password to 31 (1 reserved for string terminator) |
Yeah my main Wi-Fi password is stupid long. It's extremely old and I haven't been bothered to update all the devices on it. I thought that was the issue, the first crash report was on a guest WiFi I created with just an 8 digit password. |
Found this via Google So ssid max 31/32 I will change that in next commit and let you know (just wondering how much fun it is to enter this long password via remote control up/down buttons 🤪) |
It is painful when I can't use WPS lol. I forgot about trying my phone's hotspot, that actually was able to connect. The fixture preview worked while that was connected so I'm assuming it needs internet to work? Now that I at least have that option I can play around with custom effects/fixtures. |
It needs internet the first time, so your client browser will cache the three.js library which does the preview graphics, after that it should run without internet |
@Brandon502 password max length now 63 in latest commit, so please test if that works for you |
Seems like it fixed it. Thanks! |
Regarding when connected to PC using win11+chrome browser: I could not add wifi credentials in the GUI I just tested with a newly flashed device on w11+chrome and it worked smoothly Could you maybe describe each step you did (or retest with latest commit to see if it is still there) |
Closed as resolved |
Reported by Trackerr At April 21
Noticed the following:
when connected to PC using win11+chrome browser: I could not add wifi credentials in the GUI. After filling the fields an pressing connect, it does not connect..
when doing the same on mobile android + chrome , the wifi works as soon as i press connect.
logs attached..
See also: https://discord.com/channels/700041398778331156/1203994211301728296/1231561485495304284
The text was updated successfully, but these errors were encountered: