8000 Chrivers/apiv1 error handling rework by chrivers · Pull Request #134 · chrivers/bifrost · GitHub

More Web Proxy on the site http://driver.im/

Chrivers/apiv1 error handling rework#134

Merged

chrivers merged 11 commits intomasterfrom

chrivers/apiv1-error-handling-rework

May 9, 2025

Commits on May 9, 2025

* Make HueError::WrongType map to http status 404 (not found) to match hue bridge
chrivers
committed
- Remove obsolete HueError::UpdateUnsupported variant
chrivers
committed
+ Implement type for v1 api errors in hue crate (referenced from https://github.com/peter-murray/node-hue-api/)
chrivers
committed
* Make POST responses to /api (new user response) skip .clientkey field unless requested
chrivers
committed
* Make clip route handlers exclusively use ApiV2Result instead of "impl IntoResponse"
chrivers
committed
fixup for hueapi error
chrivers
committed
* Assign error codes as discriminants to HueApiV1Error
chrivers
committed
+ Introduce ApiV1Result and ApiV1Error, for better (actually correct) error responses in v1 api
chrivers
committed
+ Add ApiV1Error::V1CreateUnsupported variant
chrivers
committed
* Make all api v1 route handlers use ApiV1Result type, to ensure correct error mapping
chrivers
committed
- Remove obsolete ApiError::V1CreateUnsupported
chrivers
committed

0