Skip to content

fix(http): preserve Agnes error response bodies - #8

Open
jomeswang wants to merge 1 commit into
mainfrom
fix/http-error-details
Open

fix(http): preserve Agnes error response bodies#8
jomeswang wants to merge 1 commit into
mainfrom
fix/http-error-details

Conversation

@jomeswang

Copy link
Copy Markdown
Owner

Summary

  • read Agnes HTTP bodies as text first so non-JSON and empty error responses are preserved
  • parse JSON when possible and sanitize secrets before exposing non-2xx details
  • avoid misclassifying HTTP errors as retryable network failures

Validation

  • npm test (34 tests passed)
  • real Agnes API: 1080p/385-frame request returned the concrete max_num_frames: 241 error through the CLI

No credentials are persisted or included in this PR.

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.

1 participant