open-telemetry / open-telemetry/opentelemetry-cpp
[BUILD] Enable SSL for curl in bazel build environment
Open
Nobody has claimed this yet.
bug
help wanted
Stale
triage/accepted
- Dominant language
- C++
- Stars
- 1.4k
- Forks
- 632
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 75
Description
Initialy reported in PR #2415
When building with bazel, CURL is not compiled with SSL.
The following need to be fixed:
- the bazel build itself
- running at least manually the OTLP HTTP functional tests, to verify the build is correct (these tests uses https)
- consider to add the OTLP HTTP functional tests in CI, for a bazel build
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
Start by inspecting the Bazel build configuration and the OTLP HTTP functional tests mentioned in the issue. Run the OTLP HTTP tests manually with the Bazel build, including their HTTPS coverage. Done means CURL has SSL support in the Bazel environment and the tests pass; consider whether they can be added to CI.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- build-system, testing-qa
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100