Skip to content

Improve CLI HTTP error messages#490

Open
mozzieongit wants to merge 2 commits intomainfrom
improve-cli-errors
Open

Improve CLI HTTP error messages#490
mozzieongit wants to merge 2 commits intomainfrom
improve-cli-errors

Conversation

@mozzieongit
Copy link
Member

Originally, we often just printed the debug output of errors. That is very confusing and noisy. This PR changes that to chaining together the human-readable error descriptions of nested errors (relevant e.g. for tcp connection errors).

Example errors:

2026... ERROR cascade: Error: HTTP response decoding failed: expected value at line 1 column 1
2026... ERROR cascade: Error: HTTP connection failed: client error (Connect): tcp connect error: Connection refused (os error 111)
2026... ERROR cascade: Error: HTTP connection timed out
2026... ERROR cascade: Error: HTTP request failed: client error (SendRequest): connection closed before message completed

Fixes: #481

@mozzieongit mozzieongit requested a review from ximon18 March 2, 2026 15:04
@mozzieongit mozzieongit self-assigned this Mar 2, 2026
@mozzieongit mozzieongit added the enhancement New feature or request label Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Daemon HTTP 404 errors result in confusing CLI errors.

1 participant