RDK-61444 : Network Manager Plugin to support Scan Specific SSID by jincysam87 · Pull Request #306 · rdkcentral/networkmanager

jincysam87 · 2026-05-08T15:59:31Z

Reason for change: Support to scan multiple SSIDs
Test Procedure: Test wifi scan API with multiple SSIDs
Risks: Low
Signed-off-by: jincysaramma_sam@comcast.com

Sync

rdkcmf-jenkins · 2026-05-08T16:00:39Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/networkmanager/306/rdkcentral/networkmanager
Commit: 3a19acc

Report detail: gist'

Copilot

Pull request overview

This PR updates the GNOME NetworkManager WiFi scan path to support scanning with multiple SSID filters (instead of a single optional SSID string), aligning the plugin with the requirement to “scan multiple SSIDs”.

Changes:

Updated wifiScanRequest API to accept a std::vector<std::string> of SSIDs to filter.
Built a GVariant aay SSID list for nm_device_wifi_request_scan_options_async() when filters are provided; otherwise falls back to a normal scan.
Updated the GNOME proxy to pass the SSID filter list through to the WiFi manager.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File	Description
plugin/gnome/NetworkManagerGnomeWIFI.h	Changes `wifiScanRequest` signature to take a vector of SSIDs.
plugin/gnome/NetworkManagerGnomeWIFI.cpp	Implements building a multi-SSID scan options payload and triggers filtered vs unfiltered scan.
plugin/gnome/NetworkManagerGnomeProxy.cpp	Passes the SSID filter list to the updated scan request API.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

            bool wifiConnectedSSIDInfo(Exchange::INetworkManager::WiFiSSIDInfo &ssidinfo);
            bool wifiConnect(const Exchange::INetworkManager::WiFiConnectTo &ssidInfo);
-            bool wifiScanRequest(std::string ssidReq = "");
+            bool wifiScanRequest(std::vector<std::string> ssidsToFilter = {});


+            if(!ssidsToFilter.empty())
            {
-                NMLOG_INFO("starting wifi scanning .. %s", ssidReq.c_str());
+                NMLOG_INFO("Starting wifi scanning for %d SSIDs:", ssidsToFilter.size());


Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

rdkcmf-jenkins · 2026-05-08T16:06:42Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/networkmanager/306/rdkcentral/networkmanager
Commit: ec5d3a2

Report detail: gist'

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

            bool wifiConnectedSSIDInfo(Exchange::INetworkManager::WiFiSSIDInfo &ssidinfo);
            bool wifiConnect(const Exchange::INetworkManager::WiFiConnectTo &ssidInfo);
-            bool wifiScanRequest(std::string ssidReq = "");
+            bool wifiScanRequest(std::vector<std::string> ssidsToFilter = {});


+            if(!ssidsToFilter.empty())
            {
-                NMLOG_INFO("starting wifi scanning .. %s", ssidReq.c_str());
+                NMLOG_INFO("Starting wifi scanning for %d SSIDs:", ssidsToFilter.size());


Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

rdkcmf-jenkins · 2026-05-08T16:14:44Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/networkmanager/306/rdkcentral/networkmanager
Commit: 3aba5bb

Report detail: gist'

rdkcmf-jenkins · 2026-05-08T16:15:33Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/networkmanager/306/rdkcentral/networkmanager
Commit: 7032c46

Report detail: gist'

rdkcmf-jenkins · 2026-05-08T16:16:21Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/networkmanager/306/rdkcentral/networkmanager
Commit: 7032c46

Report detail: gist'

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

+            if(!ssidsToFilter.empty())
            {
-                NMLOG_INFO("starting wifi scanning .. %s", ssidReq.c_str());
+                NMLOG_INFO("Starting wifi scanning for %d SSIDs:",static_cast<int>(ssidsToFilter.size()));
                GVariantBuilder builder, array_builder;
                GVariant *options;
                g_variant_builder_init(&builder, G_VARIANT_TYPE_VARDICT);
                g_variant_builder_init(&array_builder, G_VARIANT_TYPE("aay"));
-                g_variant_builder_add(&array_builder, "@ay",
-                                    g_variant_new_fixed_array(G_VARIANT_TYPE_BYTE, (const guint8 *) ssidReq.c_str(), ssidReq.length(), 1)
-                                    );
+                for (const auto& ssid : ssidsToFilter) {
+                    if (!ssid.empty()) {


@@ -1696,23 +1696,27 @@ namespace WPEFramework
                return false;
            }
            m_isSuccess = false;
-            if(!ssidReq.empty())
+            if(!ssidsToFilter.empty())
            {
-                NMLOG_INFO("starting wifi scanning .. %s", ssidReq.c_str());
+                NMLOG_INFO("Starting wifi scanning for %d SSIDs:",static_cast<int>(ssidsToFilter.size()));
                GVariantBuilder builder, array_builder;
                GVariant *options;
                g_variant_builder_init(&builder, G_VARIANT_TYPE_VARDICT);
                g_variant_builder_init(&array_builder, G_VARIANT_TYPE("aay"));
-                g_variant_builder_add(&array_builder, "@ay",
-                                    g_variant_new_fixed_array(G_VARIANT_TYPE_BYTE, (const guint8 *) ssidReq.c_str(), ssidReq.length(), 1)
-                                    );
+                for (const auto& ssid : ssidsToFilter) {
+                    if (!ssid.empty()) {
+                        g_variant_builder_add(&array_builder, "@ay",
+                                            g_variant_new_fixed_array(G_VARIANT_TYPE_BYTE, (const guint8 *) ssid.c_str(), ssid.length(), 1)
+                                            );
+                    }
+                }
                g_variant_builder_add(&builder, "{sv}", "ssids", g_variant_builder_end(&array_builder));
                options = g_variant_builder_end(&builder);
                nm_device_wifi_request_scan_options_async(wifiDevice, options, m_cancellable, wifiScanCb, this);
                g_variant_unref(options); // Unreference the GVariant after passing it to the async function


Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

rdkcmf-jenkins · 2026-05-08T17:13:13Z

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/networkmanager/306/rdkcentral/networkmanager
Commit: 8b1f241

Report detail: gist'

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

                g_variant_builder_init(&array_builder, G_VARIANT_TYPE("aay"));
-                g_variant_builder_add(&array_builder, "@ay",
-                                    g_variant_new_fixed_array(G_VARIANT_TYPE_BYTE, (const guint8 *) ssidReq.c_str(), ssidReq.length(), 1)
-                                    );
+                for (const auto& ssid : ssidsToFilter) {
+                    g_variant_builder_add(&array_builder, "@ay",
+                                        g_variant_new_fixed_array(G_VARIANT_TYPE_BYTE, (const guint8 *) ssid.c_str(), ssid.length(), 1)
+                                        );


jincysam87 added 13 commits April 22, 2026 22:44

Merge pull request #19 from rdkcentral/develop

594597f

Sync

Merge branch 'rdkcentral:develop' into develop

712c8e8

Merge branch 'rdkcentral:develop' into develop

1f5f624

Merge branch 'rdkcentral:develop' into develop

7a0e1f8

Update NetworkManagerGnomeProxy.cpp

1fd41f2

Update NetworkManagerGnomeWIFI.cpp

3e1b457

Update NetworkManagerGnomeWIFI.h

e0ed060

Update NetworkManagerGnomeWIFI.cpp

5448555

Update NetworkManagerImplementation.cpp

17d1c07

Update NetworkManagerImplementation.cpp

c79bae1

Update NetworkManagerImplementation.cpp

bb2d211

Update NetworkManagerImplementation.cpp

96e9296

Update NetworkManagerGnomeWIFI.cpp

3a19acc

Copilot AI review requested due to automatic review settings May 8, 2026 15:59

jincysam87 requested a review from a team as a code owner May 8, 2026 15:59

Copilot started reviewing on behalf of jincysam87 May 8, 2026 16:00 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Potential fix for pull request finding

ec5d3a2

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings May 8, 2026 16:05

Copilot started reviewing on behalf of jincysam87 May 8, 2026 16:06 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Potential fix for pull request finding

216fb46

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings May 8, 2026 16:13

Update NetworkManagerGnomeWIFI.cpp

3aba5bb

Copilot started reviewing on behalf of jincysam87 May 8, 2026 16:14 View session

Update NetworkManagerGnomeWIFI.h

7032c46

Copilot AI reviewed May 8, 2026

View reviewed changes

Comment thread plugin/gnome/NetworkManagerGnomeWIFI.h

Comment thread plugin/gnome/NetworkManagerGnomeWIFI.h

Comment thread plugin/gnome/NetworkManagerGnomeWIFI.cpp

jincysam87 added 2 commits May 8, 2026 12:40

Update NetworkManagerGnomeWIFI.cpp

712eeba

Update NetworkManagerLibnmTest.cpp

dbe7119

Copilot AI review requested due to automatic review settings May 8, 2026 16:52

Copilot started reviewing on behalf of jincysam87 May 8, 2026 16:52 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Potential fix for pull request finding

9a9ff93

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings May 8, 2026 17:02

Copilot started reviewing on behalf of jincysam87 May 8, 2026 17:03 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Comment thread plugin/gnome/NetworkManagerGnomeWIFI.cpp

Comment thread plugin/gnome/NetworkManagerGnomeWIFI.cpp

Comment thread plugin/gnome/NetworkManagerGnomeProxy.cpp

jincysam87 added 2 commits May 8, 2026 13:11

Update NetworkManagerGnomeProxy.cpp

b5d5456

Update NetworkManagerGnomeWIFI.cpp

8b1f241

Copilot AI review requested due to automatic review settings May 8, 2026 17:12

Copilot started reviewing on behalf of jincysam87 May 8, 2026 17:12 View session

Copilot AI reviewed May 8, 2026

View reviewed changes

Conversation

jincysam87 commented May 8, 2026

Uh oh!

rdkcmf-jenkins commented May 8, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rdkcmf-jenkins commented May 8, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

rdkcmf-jenkins commented May 8, 2026

Uh oh!

rdkcmf-jenkins commented May 8, 2026

Uh oh!

rdkcmf-jenkins commented May 8, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rdkcmf-jenkins commented May 8, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants