Skip to content

RDKEMW-23358 : Failed to Connect to Hidden SSID - #333

Merged
karuna2git merged 1 commit into
developfrom
feature/RDKEMW-23357
Aug 11, 2026
Merged

RDKEMW-23358 : Failed to Connect to Hidden SSID#333
karuna2git merged 1 commit into
developfrom
feature/RDKEMW-23357

Conversation

@jincysam87

Copy link
Copy Markdown
Contributor

Reason for change: wifi connect call with invalid/hidden SSID results in IPC timeout
Test Procedure: Issue wifi connect does not timeout even if SSID is invalid/hidden
Risks: Medium

Signed-off-by: jincysaramma_sam@cable.comcast.com

Copilot AI lite review requested due to automatic review settings August 11, 2026 17:39
@jincysam87
jincysam87 requested a review from a team as a code owner August 11, 2026 17:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses an IPC timeout seen when attempting to connect to an invalid/hidden Wi‑Fi SSID by adjusting the GNOME NetworkManager Wi‑Fi connection flow when the SSID is not present in the scan list.

Changes:

  • When an SSID is not found in scan results (and persist is enabled), the code now calls deleteClientConnection() before attempting to add the SSID to known networks.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread plugin/gnome/NetworkManagerGnomeWIFI.cpp
@karuna2git
karuna2git merged commit 6c74d8e into develop Aug 11, 2026
15 checks passed
@karuna2git
karuna2git deleted the feature/RDKEMW-23357 branch August 11, 2026 18:04
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 11, 2026
@jincysam87 jincysam87 changed the title RDKEMW-23357 : Failed to Connect to Hidden SSID RDKEMW-23358 : Failed to Connect to Hidden SSID Aug 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants