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

Remove undocumented and uncontrollable WiFi RSSI filter. #5393

Merged
merged 1 commit into from
Jul 16, 2021

Conversation

rtrbt
Copy link
Contributor

@rtrbt rtrbt commented Jul 16, 2021

Commit d15e1b0e in the 2.0.0 pull request #4996 introduced a filter to connect only to WiFi networks with a RSSI of -75 or better. This results in strange behavior, as a scan still shows networks, that can't be connected to, even though with older versions, connecting to those networks was possible and the connection (albeit slow) was stable.

Remove the RSSI filter for now by setting the threshold to -127, i.e. the lowest possible value. Maybe in the future the filter threshold could be exposed to users, to allow filtering out nearly unreachable networks.

Commit d15e1b0 [1] in the 2.0.0 pull request espressif#4996 introduced a filter
to connect only to WiFi networks with a RSSI of -75 or better. This
results in strage behaviour, as a scan still shows networks, that can't
be connected to, even though with older versions, connecting to those
networks was possible and the connection (albeit slow) was stable.

Remove the RSSI filter for now by setting the threshold to -127, i.e.
the lowest possible value. Maybe in the future the filter threshold
could be exposed to users, to allow filtering out nearly unreachable
networks.

[1] github.com/espressif/pull/4996/commits/d15e1b0
@CLAassistant
Copy link

CLAassistant commented Jul 16, 2021

CLA assistant check
All committers have signed the CLA.

@me-no-dev me-no-dev merged commit 780588d into espressif:master Jul 16, 2021
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

Successfully merging this pull request may close these issues.

4 participants