Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Bump log4j from 2.20.0 to 2.24.1 #6774

Closed
wants to merge 1 commit into from

Conversation

bowenliang123
Copy link
Contributor

🔍 Description

Issue References 🔗

This pull request fixes #

Describe Your Solution 🔧

Types of changes 🔖

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪

Behavior Without This Pull Request ⚰️

Behavior With This Pull Request 🎉

Related Unit Tests


Checklist 📝

Be nice. Be informative.

@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (1d668ea) to head (98a89ef).
Report is 11 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6774   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         684     684           
  Lines       42354   42354           
  Branches     5776    5776           
======================================
  Misses      42354   42354           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yaooqinn yaooqinn added this to the v1.11.0 milestone Nov 14, 2024
@yaooqinn yaooqinn closed this in d22ad7f Nov 14, 2024
@pan3793
Copy link
Member

pan3793 commented Nov 26, 2024

2.24.2 claims to fix a critical bug, we'd better move forward

@bowenliang123
Copy link
Contributor Author

2.24.2 claims to fix a critical bug, we'd better move forward

Let's do it.

@bowenliang123 bowenliang123 deleted the log4j-2.24 branch November 26, 2024 06:53
pan3793 pushed a commit that referenced this pull request Nov 28, 2024
### Why are the changes needed?
As  pan3793  pointed out that version 2.24.1 had critical bug in #6774 (comment)
Bump bump log4j from 2.24.1 to 2.24.2
- Release:https://logging.apache.org/log4j/2.x/release-notes.html#release-notes-2-24-2

### How was this patch tested?

GA.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #6826 from pionCham/bump-log4j-version.

Closes #6826

150713f [chengpeiming] Bump log4j to 2.24.2

Lead-authored-by: chengpeiming <[email protected]>
Co-authored-by: chengpeiming <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants