Skip to content

Commit

Permalink
Replace tarpc dependency with directly using tokio-serde
Browse files Browse the repository at this point in the history
While tarpc works nicely, it pulls in a number of dependencies through
its opentelemetry support, which prevents packaging the event-broker
in Fedora.  This switches to using tokio-serde directly for RPC.

Signed-off-by: Daiki Ueno <[email protected]>
  • Loading branch information
ueno committed Nov 18, 2023
1 parent 9ef124e commit 695e8fb
Show file tree
Hide file tree
Showing 9 changed files with 80 additions and 272 deletions.
192 changes: 2 additions & 190 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -213,4 +213,3 @@ You can open the generated `flamegraph.html` with your browser.

- [libbpf-async](https://github.com/fujita/libbpf-async) for asynchronous BPF ringbuf implementation over libbpf-rs
- [rust-keylime](https://github.com/keylime/rust-keylime/) for permissions management code
- [tarpc](https://github.com/google/tarpc) for the pubsub example implementation
2 changes: 1 addition & 1 deletion crypto-auditing/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,11 @@ libc = "0.2"
serde = { version = "1.0", features = ["derive"] }
serde_cbor = "0.11"
serde_with = "2.2"
tarpc = { version = "0.33", features = ["serde-transport", "unix"] }
thiserror = "1.0"
tokio = "1.23"
tokio-serde = { version = "0.8", features=["cbor"] }
tokio-stream = "0.1"
tokio-util = { version = "0.7", features=["codec"] }
tracing = "0.1"

[build-dependencies]
Expand Down
2 changes: 0 additions & 2 deletions crypto-auditing/src/event_broker.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
mod error;
pub use error::{Error, Result};

mod service;

mod client;
pub use client::{Client, ClientHandle};

Expand Down
Loading

0 comments on commit 695e8fb

Please sign in to comment.