Merge branch '3.3.x'

See gh-41892

Author: snicoll

spring-boot-project/spring-boot-dependencies/build.gradle

@@ -1141,7 +1141,7 @@ bom {
 			releaseNotes("https://github.com/apache/logging-log4j2/releases/tag/rel%2F{version}")
 		}
 	}
-	library("Logback", "1.5.7") {
+	library("Logback", "1.5.6") {
 		group("ch.qos.logback") {
 			modules = [
 				"logback-classic",

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/logback/DefaultLogbackConfiguration.java

@@ -17,7 +17,6 @@
 package org.springframework.boot.logging.logback;
 
 import java.nio.charset.Charset;
-import java.util.concurrent.locks.ReentrantLock;
 
 import ch.qos.logback.classic.Level;
 import ch.qos.logback.classic.encoder.PatternLayoutEncoder;
@@ -52,7 +51,6 @@
  * @author Scott Frederick
  * @author Jonatan Ivanov
  * @author Moritz Halbritter
- * @author Mark Chesney
  */
 class DefaultLogbackConfiguration {
 
@@ -82,9 +80,7 @@ class DefaultLogbackConfiguration {
 	}
 
 	void apply(LogbackConfigurator config) {
-		ReentrantLock lock = config.getConfigurationLock();
-		lock.lock();
-		try {
+		synchronized (config.getConfigurationLock()) {
 			defaults(config);
 			Appender<ILoggingEvent> consoleAppender = consoleAppender(config);
 			if (this.logFile != null) {
@@ -95,9 +91,6 @@ void apply(LogbackConfigurator config) {
 				config.root(Level.INFO, consoleAppender);
 			}
 		}
-		finally {
-			lock.unlock();
-		}
 	}
 
 	private void defaults(LogbackConfigurator config) {

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/logback/LogbackConfigurator.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2024 the original author or authors.
+ * Copyright 2012-2023 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,7 +18,6 @@
 
 import java.util.HashMap;
 import java.util.Map;
-import java.util.concurrent.locks.ReentrantLock;
 
 import ch.qos.logback.classic.Level;
 import ch.qos.logback.classic.Logger;
@@ -50,7 +49,7 @@ LoggerContext getContext() {
 		return this.context;
 	}
 
-	ReentrantLock getConfigurationLock() {
+	Object getConfigurationLock() {
 		return this.context.getConfigurationLock();
 	}
 

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/LogbackLoggingSystemParallelInitializationTests.java

@@ -45,7 +45,6 @@ class LogbackLoggingSystemParallelInitializationTests {
 	void cleanUp() {
 		this.loggingSystem.cleanUp();
 		((LoggerContext) LoggerFactory.getILoggerFactory()).stop();
-		((LoggerContext) LoggerFactory.getILoggerFactory()).reset();
 	}
 
 	@Test

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/LogbackLoggingSystemTests.java

@@ -127,7 +127,6 @@ void cleanUp() {
 		System.getProperties().keySet().retainAll(this.systemPropertyNames);
 		this.loggingSystem.cleanUp();
 		((LoggerContext) LoggerFactory.getILoggerFactory()).stop();
-		((LoggerContext) LoggerFactory.getILoggerFactory()).reset();
 	}
 
 	@Test

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/SpringBootJoranConfiguratorTests.java

@@ -72,7 +72,6 @@ void setup(CapturedOutput output) {
 	@AfterEach
 	void reset() {
 		this.context.stop();
-		this.context.reset();
 		new BasicConfigurator().configure((LoggerContext) LoggerFactory.getILoggerFactory());
 	}