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

Wifi scan shows wrong BSSIDs #1567

Closed
a-tom-s opened this issue Feb 22, 2019 · 1 comment
Closed

Wifi scan shows wrong BSSIDs #1567

a-tom-s opened this issue Feb 22, 2019 · 1 comment
Milestone

Comments

@a-tom-s
Copy link
Contributor

a-tom-s commented Feb 22, 2019

Bug description
I noticed that the BSSIDs in the Wifi scan are different from the BSSIDs of my devices, but I realized the correct BSSIDs are part of the shown.
It's caused because in wifi.ino / _wifiScan line.119 the BSSID string array starts with BSSID[1] instead of BSSID[0].

Steps to reproduce
Start SCAN on the Wifi page and compare the detected BSSID with the BSSID of your AP.
On the Status page you can see the correct BSSID of your device, here it is shown correctly.

@mcspr
Copy link
Collaborator

mcspr commented Feb 23, 2019

huh. scan results are from justwifi, where it is correctly parsed:
https://github.com/xoseperez/justwifi/blob/8bb2c16f5b91f56de191cb8ff96a816fb672cfd2/src/JustWifi.cpp#L188-L200
here, it is not though:

BSSID_scan[1], BSSID_scan[2], BSSID_scan[3], BSSID_scan[4], BSSID_scan[5], BSSID_scan[6],

maybe extend justwifi a bit so it gives more scanning info via more arguments? not that repetition is bad, but imo that could simplify the process a lot.
and it'll show hidden networks too?

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

No branches or pull requests

3 participants