Bug: Missing curl_error() handling across all service plugins #123

New Issue

2026-06-06T11:50:11Z

jmiller commented

2026-06-06 11:50:11 +00:00

Audit Finding

Severity: Low

Issue:
No service plugin checks curl_errno() or curl_error() after curl_exec(). DNS failures, SSL errors, and timeouts silently produce empty results with no useful error message.

Fix: After curl_exec(), check for false and capture curl_error().

Files: All service plugins

## Audit Finding **Severity:** Low **Issue:** No service plugin checks `curl_errno()` or `curl_error()` after `curl_exec()`. DNS failures, SSL errors, and timeouts silently produce empty results with no useful error message. **Fix:** After `curl_exec()`, check for `false` and capture `curl_error()`. **Files:** All service plugins

jmiller commented

2026-06-06 11:50:16 +00:00

Branch created: feature/123-bug-missing-curl-error-handling-across-a

git fetch origin
git checkout feature/123-bug-missing-curl-error-handling-across-a

Branch created: [`feature/123-bug-missing-curl-error-handling-across-a`](https://git.mokoconsulting.tech/MokoConsulting/MokoJoomCross/src/branch/feature/123-bug-missing-curl-error-handling-across-a) ```bash git fetch origin git checkout feature/123-bug-missing-curl-error-handling-across-a ```

jmiller referenced this issue from a commit

2026-06-06 14:42:01 +00:00

fix: resolve remaining low-priority bugs (#121, #122, #123, #126)

jmiller closed this issue

2026-06-06 14:42:26 +00:00

Sign in to join this conversation.

Type —

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/MokoJoomCross#123