Skip to content

fix(wifi): support open STA network and improve diagnostics#117

Merged
wujiangang merged 1 commit into
espressif:masterfrom
leeebo:fix/wifi-open-network
Jun 5, 2026
Merged

fix(wifi): support open STA network and improve diagnostics#117
wujiangang merged 1 commit into
espressif:masterfrom
leeebo:fix/wifi-open-network

Conversation

@leeebo

@leeebo leeebo commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Description

This PR fixes the main Wi-Fi issue for open STA networks and improves startup diagnostics.

Changes

  • Fix open-network STA connection by setting sta_cfg.sta.threshold.authmode to WIFI_AUTH_OPEN when wifi_password is empty.
  • Keep WPA2 threshold for password-protected networks.
  • Validate STA SSID/password length in app config validation and wifi manager validation to prevent silent truncation and invalid runtime apply.
  • Improve startup connection result semantics by distinguishing retry-exhausted failure from timeout.
  • Add sanitized startup/disconnect/fallback logs to improve field diagnostics.
  • Update changelog entry for 2026-06-04 to reflect the open-network fix as the primary issue.

Testing

  • pre-commit run

Related

Fix the main Wi-Fi issue for open networks by setting the STA auth threshold to WIFI_AUTH_OPEN when the password is empty.

- support open-network STA connection instead of always requiring WPA2
- validate STA SSID/password length in app config and wifi manager
- distinguish retry-exhausted failures from timeout in startup wait handling
- add sanitized startup/disconnect/fallback logs for observability

Related to espressif#110
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@laride

laride commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

sha=fa35df6948cffacc454b51f9f784f5208b83d661

@wujiangang
wujiangang merged commit 74d851d into espressif:master Jun 5, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants