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

Open Wifi networks not supported #6

Closed
stigwoxholt opened this issue Dec 9, 2018 · 4 comments
Closed

Open Wifi networks not supported #6

stigwoxholt opened this issue Dec 9, 2018 · 4 comments

Comments

@stigwoxholt
Copy link

If i select a network that is open i get prompted to enter the password. It is not possible to get past this screen without entering a password. If i enter something and move on, it get a message saying i entered the wrong password.

Expected :

  1. Network list should show open networks ( it does today ) but add a small "open network icon" to them
  2. When i select a open network i am not prompted to enter password, just a question if i want to connect.
@t-mon
Copy link
Member

t-mon commented Dec 25, 2018

Hi stigwoxholt. Thanks for your feedback!
The open access point feature is still missing, because this tool is an excerpt from an actual product, which should never allow open wifi networks for security reasons. We created an open source tool out of it and the feature is on the roadmap! :)

@maxpage92
Copy link

Hi there, if I wanted to add support for not needing a wifi password in the app, where would I change that? I have a use case, where many users won't have a wifi password set but still need to be able to connect. Any tips would be much appreciated :) thanks

@mzanetti
Copy link
Member

Well, there's 2 places. One is nymea-networkmanager, which runs on the Raspberry Pi and receives the WiFi login detail, and the other is the Berrylan app.
(In fact, there's 3 places, the BT-WiFi-Setup is also supported in nymea:app...)

If want to get going with contributing this, I'd recommend to first build BerryLan for your phone. Once that works, change it to allow setting up wifis without password.
Start here: https://github.com/nymea/berrylan/blob/master/main.qml#L232
If the wifi does not require authentication (!d.currentAP.isProtected) instead of just going to the next page with swipeView.currentIndex++, you'd do the connectWirelessNetwork call right away with an empty password and skip the password page.

If you're lucky, nymea-networkmanager accepts it just like that. If not you'd then need to checkout the nymea-networkmanager/libnymea-networkmanager sources and change things there along the way until it does in a similar fashion.

@mzanetti
Copy link
Member

mzanetti commented Nov 7, 2019

I've added support for open wifis in the berrylan app, it'll be released with the next update. However, there is indeed something to be fixed in nymea-networkmanager too.

I'm closing this issue here now as it should work fine from an app perspective now. Please follow this bug to watch the issue in nymea-networkmanager: nymea/nymea-networkmanager#22

@mzanetti mzanetti closed this as completed Nov 7, 2019
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

4 participants