(WIP) ekump/APMSP-2516 implement http common component#1624
Draft
(WIP) ekump/APMSP-2516 implement http common component#1624
Conversation
ekump
changed the title
Clippy Allow Annotation ReportComparing clippy allow annotations between branches:
Summary by Rule
Annotation Counts by File
Annotation Stats by Crate
About This ReportThis report tracks Clippy allow annotations for specific rules, showing how they've changed in this PR. Decreasing the number of these annotations generally improves code quality. |
📚 Documentation Check Results📦
|
🔒 Cargo Deny Results✅ No issues found! 📦
|
ekump
force-pushed
the
ekump/APMSP-2516-implement-http-common-component
branch
2 times, most recently
from
919247e to
0c989d9
Compare
BenchmarksComparisonBenchmark execution time: 2026-02-26 13:30:29 Comparing candidate commit 5d99312 in PR branch Found 2 performance improvements and 0 performance regressions! Performance is the same for 55 metrics, 2 unstable metrics. scenario:normalization/normalize_service/normalize_service/[empty string]
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
Group 16
Group 17
Group 18
Group 19
BaselineOmitted due to size. |
Contributor
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1624 +/- ##
==========================================
+ Coverage 71.19% 71.34% +0.14%
==========================================
Files 423 432 +9
Lines 62453 63026 +573
==========================================
+ Hits 44466 44964 +498
- Misses 17987 18062 +75
🚀 New features to boost your workflow:
|
ekump
force-pushed
the
ekump/APMSP-2516-implement-http-common-component
branch
2 times, most recently
from
9093e6a to
5570b43
Compare
|
✨ Fix all issues with BitsAI or with Cursor
|
Contributor
|
It might make sense to tie this into the ddcommon |
ekump
force-pushed
the
ekump/APMSP-2516-implement-http-common-component
branch
from
5570b43 to
c1327a8
Compare
Contributor
Author
|
@danielsn @brettlangdon did mention that it would be nice if there was an "agent client". I think it makes sense. We can build that on top of this generic http client. I assume we'll be able to extract logic in Endpoint so it's shared (or just use it directly). |
New HTTP client crate for use by external language clients via FFI. Defines HttpClient, HttpRequest, HttpResponse, HttpClientError, and a builder API with configurable timeout, transport, and retry.
ReqwestBackend sends requests via reqwest::Client with connection pooling, error mapping, and per-request timeout. Integration tests cover HTTP round-trips, timeout behavior, and connection pooling.
Builder API supports unix_socket() and windows_named_pipe() for platform-specific transports. FIPS TLS via rustls-no-provider with init_fips_crypto() helper for callers to install the crypto provider.
Opt-in retry via RetryConfig on the builder. Supports configurable max retries, initial delay, exponential backoff, and jitter. All errors except InvalidConfig are retried.
Add MultipartPart type and multipart_parts field on HttpRequest. When parts are present, the request is sent as multipart/form-data with reqwest handling Content-Type and boundary automatically. Setting both multipart parts and a body returns an error.
Add a hyper-backend feature as a lighter alternative to reqwest-backend for consumers where binary size matters. The hyper backend uses libdd-common's Connector and a new shared multipart encoder in libdd-common. CI runs tests against both backends.
When both reqwest-backend and hyper-backend are enabled (e.g. CI with --all-features), reqwest takes precedence. Hyper backend only activates when reqwest-backend is not enabled.
ekump
force-pushed
the
ekump/APMSP-2516-implement-http-common-component
branch
from
0a1ac56 to
5d99312
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
A generic HTTP client that can be used by other languages.
NOTE: This PR will be updated to work with the shared runtime component as soon as it lands before being ready for proper review because 🍴
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.