Skip to content

Fix log message in ApiLogger::setUseParentHandlers#3943

Merged
vy merged 1 commit intoapache:2.xfrom
jmax01:bugfix/#3942-replace-setLevel-with-setUseParentHandlers-in-ApiLogger.setUseParentHandlers
Oct 9, 2025
Merged

Fix log message in ApiLogger::setUseParentHandlers#3943
vy merged 1 commit intoapache:2.xfrom
jmax01:bugfix/#3942-replace-setLevel-with-setUseParentHandlers-in-ApiLogger.setUseParentHandlers

Conversation

@jmax01
Copy link
Copy Markdown
Contributor

@jmax01 jmax01 commented Sep 26, 2025 •
edited by vy
Loading

replace setLevel with setUseParentHandlers

Fixes #3942

@jmax01 jmax01 changed the title Bugfix/#3942 replace set level with set use parent handlers in api logger.set use parent handlers Bugfix/#3942 replace set level with setUseParentHandlers in ApiLogger.setUseParentHandlers Sep 26, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 27, 2025 •
edited
Loading

Job Requested goals Build Tool Version Build Outcome Build Scan®
build-macos-latest clean install 3.9.8 ✅ Build Scan PUBLISHED
build-ubuntu-latest clean install 3.9.8 ✅ Build Scan PUBLISHED
build-windows-latest clean install 3.9.8 ✅ Build Scan PUBLISHED
Generated by gradle/develocity-actions

@vy
Copy link
Copy Markdown
Member

vy commented Oct 3, 2025

@jmax01, thanks for the fix. Would you mind signing all your commits, please? You'll need to force-push at the end.

See GitHub Docs on Commit Signature Verification.

@vy vy self-assigned this Oct 3, 2025
@jmax01
Copy link
Copy Markdown
Contributor Author

jmax01 commented Oct 7, 2025

@jmax01, thanks for the fix. Would you mind signing all your commits, please? You'll need to force-push at the end.

See GitHub Docs on Commit Signature Verification.

Was out of town, will try to get this knocked out today.

@jmax01
apache#3942 Fix log message in setUseParentHandlers method
92684f5 
Replace `setLevel` with `setUseParentHandlers`.
@jmax01 jmax01 force-pushed the bugfix/#3942-replace-setLevel-with-setUseParentHandlers-in-ApiLogger.setUseParentHandlers branch from b4e0544 to 92684f5 Compare October 9, 2025 00:56
@jmax01
Copy link
Copy Markdown
Contributor Author

jmax01 commented Oct 9, 2025

@jmax01, thanks for the fix. Would you mind signing all your commits, please? You'll need to force-push at the end.

See GitHub Docs on Commit Signature Verification.
Should be ok now.

@vy vy changed the title Bugfix/#3942 replace set level with setUseParentHandlers in ApiLogger.setUseParentHandlers Fix log message in ApiLogger::setUseParentHandlers Oct 9, 2025
@vy vy enabled auto-merge (squash) October 9, 2025 07:03
@vy vy merged commit 8a3fb53 into apache:2.x Oct 9, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from To triage to Done in Log4j bug tracker Oct 9, 2025
vy pushed a commit that referenced this pull request Dec 15, 2025
@jmax01 @vy
Fix log message in ApiLogger::setUseParentHandlers (#3943)
4bfb155
vy pushed a commit that referenced this pull request Dec 15, 2025
@jmax01 @vy
Fix log message in ApiLogger::setUseParentHandlers (#3943)
73db4fb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vy vy vy approved these changes

Assignees

@vy vy

Labels

None yet

Projects

Log4j bug tracker
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ApiLogger.setUseParentHandlers warning references setLevel and not setUseParentHandlers

2 participants

@jmax01 @vy