Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat.Req] Add HTTP request compression to RAV requests #97

Open
aasseman opened this issue Nov 14, 2023 · 0 comments
Open

[Feat.Req] Add HTTP request compression to RAV requests #97

aasseman opened this issue Nov 14, 2023 · 0 comments
Assignees
Labels
p2 Medium priority size:small Small type:feature New or enhanced functionality

Comments

@aasseman
Copy link
Collaborator

Problem statement
The TAP Aggregator API is based on JSON-RPC. Thus there are easy bandwidth gains to be had with request compression since JSON is usually highly compressible.

Additional context

// TODO: Request compression and response decompression. Also a fancy user agent?
let client = HttpClientBuilder::default()
.request_timeout(Duration::from_secs(
inner.config.tap.rav_request_timeout_secs,
))
.build(&inner.sender_aggregator_endpoint)?;

@aasseman aasseman added size:small Small p2 Medium priority type:feature New or enhanced functionality labels Nov 14, 2023
@aasseman aasseman added this to the TAP Agent ready for prod milestone Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2 Medium priority size:small Small type:feature New or enhanced functionality
Projects
None yet
Development

No branches or pull requests

2 participants