open-telemetry / open-telemetry/opentelemetry-cpp
Evaluate removing curl dependency for Apple platform - Adding support for CFNetwork API
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 1.4k
- Forks
- 632
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 75
Description
Apple comes with CFNetwork API to enable applications to interact with FTP and HTTP protocols to access Internet resources. This means it doesn't explicitly need to use curl as an HTTP client library.
otel-cpp is designed to plugin any external client library by extending abstract classes defined in the ext::http::client namespace.
And HttpClientFactory factory is responsible to pick up the correct library based on the platform it is running.
CURL implementation is available here: https://github.com/open-telemetry/opentelemetry-cpp/tree/main/ext/include/opentelemetry/ext/http/client/curl
Similar implementation needs to be provided to support CFNetwork API.
This issue is created to evaluate whether CFNetwork can be supported, and probably raise another issue for the implementation.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Review ext/include/opentelemetry/ext/http/client/curl and the ext::http::client abstractions, then trace HttpClientFactory's platform selection. Compare the required CFNetwork integration with those interfaces and record whether it can replace curl on Apple platforms; if viable, the done state is a follow-up implementation issue with its scope defined.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- networking, operating-systems
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100