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

Add ability to enter desired BSSID #4429

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

blazoncek
Copy link
Collaborator

@blazoncek blazoncek commented Dec 25, 2024

  • add event handling (for debug purposes and correct status setting)
  • fixes WiFi connection by BSSID #2151
  • moved WLED::initEthernet() and WLED::findWiFi() to network.cpp
  • added string conversion functions for MAC address (without colons)
  • used MAC conversion functions for configuration
  • replaced WLED_WIFI_CONFIGURED with function call (and updated macro)
  • fixed correct use of WLED_ETHERNET
  • added AP client counter (not really used currently but it will get its meaning soon if you stop bugging me with updating PR descriptions) 😉

Oh, and BTW, there is a bug in src/dependencies/network/Network.cpp.

- add event handling (debug)
- fixes Aircoookie#2151
@blazoncek blazoncek added enhancement connectivity Issue regarding protocols, WiFi connection or availability of interfaces labels Dec 25, 2024
Copy link

@Liliputech Liliputech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good change overall with a nice move of initEthernet from wled.cpp to network.cpp.
Although most compilations (jobs) are successful, some of them have warnings (not breaking though)
Some of them seems to be related to your patch / code move.

@blazoncek
Copy link
Collaborator Author

That's just a harbinger of things to come. 😉

@netmindz
Copy link
Collaborator

Please update the PR title to fully summarise the changes contained and extend the description as well please @blazoncek

@blazoncek
Copy link
Collaborator Author

blazoncek commented Dec 26, 2024

@netmindz the gist of this PR is in its title - ability to force BSSID selection. All other changes are just collateral damage from my fork. I do not intend to remove those as it is too much hassle otherwise.

EDIT: The bugging part is a joke.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connectivity Issue regarding protocols, WiFi connection or availability of interfaces enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WiFi connection by BSSID
3 participants