Skip to content

Drop tonic dependency#46

Merged
declark1 merged 2 commits intomainfrom
drop-tonic
Apr 17, 2025
Merged

Drop tonic dependency#46
declark1 merged 2 commits intomainfrom
drop-tonic

Conversation

@declark1
Copy link
Contributor

@declark1 declark1 commented Apr 17, 2025

This PR makes changes required to drop tonic as a dependency from mocktail. It is still used as a client in mocktail-tests.

Changes:

  • Add status.rs
    • Move StatusCode here
    • Add Code, based on tonic::Code (with http code mapping changes)
  • Drop CodeExt
  • Update gRPC service:
    • Use BoxBody (with hyper::Error) instead of tonic::body::BoxBody and associated empty() body helper instead of tonic::body::empty_body()
    • Use hyper::Error in Result instead of tonic::Status
    • Use Code for grpc-status header instead of tonic::Code

Closes #36
Closes #43

Signed-off-by: declark1 <44146800+declark1@users.noreply.github.com>
Signed-off-by: declark1 <44146800+declark1@users.noreply.github.com>
@declark1 declark1 requested a review from mdevino April 17, 2025 17:27
@declark1 declark1 merged commit d1825ab into main Apr 17, 2025
2 checks passed
@declark1 declark1 deleted the drop-tonic branch April 17, 2025 17:37
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.

Latest alpha fails to build Drop tonic dependency

1 participant