Skip to content

Fix misleading "curl init failed" error log in Device_WiFi_SSID.cpp - #405

Merged
Vismalskumar0 merged 2 commits into
feature/RDKEMW-14686_finalfrom
copilot/sub-pr-399-please-work
Mar 12, 2026
Merged

Fix misleading "curl init failed" error log in Device_WiFi_SSID.cpp#405
Vismalskumar0 merged 2 commits into
feature/RDKEMW-14686_finalfrom
copilot/sub-pr-399-please-work

Conversation

Copilot AI commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

The else branch handling an empty getJsonRPCData() response for GetAvailableInterfaces was logging "curl init failed" — a message that implies a cURL initialization error rather than an empty/failed JSON-RPC response.

Change

Updated the error message to "getJsonRPCData() failed", consistent with the equivalent empty-response else branch already used for the GetWifiState call:

// Before
RDK_LOG(RDK_LOG_ERROR, LOG_TR69HOSTIF, "%s: curl init failed\n", __FUNCTION__);

// After
RDK_LOG(RDK_LOG_ERROR, LOG_TR69HOSTIF, "%s: getJsonRPCData() failed\n", __FUNCTION__);

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: Vismalskumar0 <188226757+Vismalskumar0@users.noreply.github.com>
@Vismalskumar0
Vismalskumar0 marked this pull request as ready for review March 12, 2026 10:56
@Vismalskumar0
Vismalskumar0 requested a review from a team as a code owner March 12, 2026 10:56
Copilot AI review requested due to automatic review settings March 12, 2026 10:56
Copilot AI changed the title [WIP] [WIP] Address feedback on Device_WiFi_SSID.cpp updates Fix misleading "curl init failed" error log in Device_WiFi_SSID.cpp Mar 12, 2026
@Vismalskumar0
Vismalskumar0 merged commit 2d0e71b into feature/RDKEMW-14686_final Mar 12, 2026
1 check passed
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 12, 2026
Copilot AI removed the request for review from Copilot March 23, 2026 22:43
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.

2 participants