Skip to content

fix: classify CI token refresh communication failures#546

Merged
facundo-herodevs merged 1 commit intomainfrom
757-handle-null-response
Apr 15, 2026
Merged

fix: classify CI token refresh communication failures#546
facundo-herodevs merged 1 commit intomainfrom
757-handle-null-response

Conversation

@facundo-herodevs
Copy link
Copy Markdown
Member

@facundo-herodevs facundo-herodevs commented Apr 15, 2026

Closes https://github.com/neverendingsupport/data-and-integrations/issues/757

What This Branch Does

This branch improves CI-token refresh failures in the CLI so null, malformed, or transport-level HeroDevs API responses are no longer surfaced as “CI token is invalid or expired.” It adds explicit communication-failure classification in the CI token client, maps those failures to a new CLI-facing error code/message, and keeps scan eol output user-facing instead of exposing internal error class names.

CI Token Refresh Classification

CLI Messaging

@facundo-herodevs facundo-herodevs requested a review from a team as a code owner April 15, 2026 17:26
@facundo-herodevs facundo-herodevs merged commit 8abaee6 into main Apr 15, 2026
12 checks passed
@facundo-herodevs facundo-herodevs deleted the 757-handle-null-response branch April 15, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants