Skip to content

fix: remove invalid .cause() call that breaks compilation - #20

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
v1.4.0from
fix/remove-invalid-cause
Aug 19, 2026
Merged

fix: remove invalid .cause() call that breaks compilation#20
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
v1.4.0from
fix/remove-invalid-cause

Conversation

@dengkliu92

Copy link
Copy Markdown

The .cause(e) call on line 1103 fails to compile because anyhow::Error doesn't implement std::error::Error in the way DynamoErrorBuilder::cause expects:

error[E0277]: the trait bound `anyhow::Error: std::error::Error` is not satisfied

The error message already includes the full error chain via {e:#}, so dropping .cause() doesn't lose any information. The fix still correctly converts template validation errors to InvalidArgument (400 status).

Fixes build failure in dynamo-vllm-runtime and other 1.4.0 images.

The .cause(e) call on line 1103 fails to compile because anyhow::Error
doesn't implement std::error::Error in the way DynamoErrorBuilder::cause
expects:

    error[E0277]: the trait bound `anyhow::Error: std::error::Error` is not satisfied

The error message already includes the full error chain via {e:#}, so
dropping .cause() doesn't lose any information. The fix still correctly
converts template validation errors to InvalidArgument (400 status).

Fixes build failure in dynamo-vllm-runtime and other 1.4.0 images.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant