Skip to content

Conversation

@marcalff
Copy link
Member

Fixes #2357

Changes

Please provide a brief description of the changes here.

  • In the bazel build, upgrade to curl 8.4.0
  • The CMake build is not affected

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@marcalff marcalff requested a review from a team October 11, 2023 08:40
@marcalff marcalff added pr:please-review This PR is ready for review CVE Common Vulnerabilities and Exposures labels Oct 11, 2023
@marcalff marcalff mentioned this pull request Oct 11, 2023
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #2358 (6634e9e) into main (18a27df) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2358      +/-   ##
==========================================
+ Coverage   87.41%   87.43%   +0.02%     
==========================================
  Files         199      199              
  Lines        6018     6018              
==========================================
+ Hits         5260     5261       +1     
+ Misses        758      757       -1     

see 1 file with indirect coverage changes

@marcalff marcalff merged commit df96b74 into open-telemetry:main Oct 11, 2023
@marcalff marcalff deleted the fix_curl_security_2357 branch October 27, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CVE Common Vulnerabilities and Exposures pr:please-review This PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

upgrade libcurl for security vulnerabilities.

2 participants