Skip to content
New issue

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

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

Already on GitHub? # to your account

[fix][sec] Upgrade protobuf-java to 3.25.5 #23356

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Sep 26, 2024

Main Issue: #23341

Motivation

Modifications

  • Upgrade protobuf-java to 3.25.5, this PR is currently WIP. The goal is to see if our CI passes.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.49%. Comparing base (bbc6224) to head (b4778a1).
Report is 606 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #23356      +/-   ##
============================================
+ Coverage     73.57%   74.49%   +0.92%     
- Complexity    32624    34428    +1804     
============================================
  Files          1877     1934      +57     
  Lines        139502   145028    +5526     
  Branches      15299    15846     +547     
============================================
+ Hits         102638   108038    +5400     
+ Misses        28908    28707     -201     
- Partials       7956     8283     +327     
Flag Coverage Δ
inttests 27.45% <ø> (+2.87%) ⬆️
systests 24.58% <ø> (+0.26%) ⬆️
unittests 73.84% <ø> (+1.00%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 602 files with indirect coverage changes

@lhotari lhotari added the release/blocker Indicate the PR or issue that should block the release until it gets resolved label Sep 27, 2024
@lhotari lhotari requested a review from merlimat September 30, 2024 16:02
@merlimat merlimat merged commit ab684a0 into apache:master Oct 3, 2024
61 of 63 checks passed
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Oct 4, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Oct 4, 2024
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Oct 4, 2024
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Oct 4, 2024
hanmz pushed a commit to hanmz/pulsar that referenced this pull request Feb 12, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/security cherry-picked/branch-3.0 cherry-picked/branch-3.3 doc-not-needed Your PR changes do not impact docs ready-to-test release/blocker Indicate the PR or issue that should block the release until it gets resolved release/3.0.7 release/3.3.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants