diff --git a/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/main/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHost.java b/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/main/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHost.java index 5c21405087..ddaab19933 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/main/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHost.java +++ b/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/main/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHost.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2023 the original author or authors. + * Copyright 2012-2024 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. diff --git a/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/test/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHostTests.java b/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/test/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHostTests.java index a2949d650c..a2b8de3f7c 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/test/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHostTests.java +++ b/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/src/test/java/org/springframework/boot/buildpack/platform/docker/configuration/ResolvedDockerHostTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2023 the original author or authors. + * Copyright 2012-2024 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. diff --git a/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/logback/LogbackLoggingSystem.java b/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/logback/LogbackLoggingSystem.java index 332b92d92b..e30da9139b 100644 --- a/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/logback/LogbackLoggingSystem.java +++ b/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/logback/LogbackLoggingSystem.java @@ -295,14 +295,9 @@ public class LogbackLoggingSystem extends AbstractLoggingSystem implements BeanF private void configureByResourceUrl(LoggingInitializationContext initializationContext, LoggerContext loggerContext, URL url) throws JoranException { - if (url.getPath().endsWith(".xml")) { - JoranConfigurator configurator = new SpringBootJoranConfigurator(initializationContext); - configurator.setContext(loggerContext); - configurator.doConfigure(url); - } - else { - throw new IllegalArgumentException("Unsupported file extension in '" + url + "'. Only .xml is supported"); - } + JoranConfigurator configurator = new SpringBootJoranConfigurator(initializationContext); + configurator.setContext(loggerContext); + configurator.doConfigure(url); } private void stopAndReset(LoggerContext loggerContext) { diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/LogbackLoggingSystemTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/LogbackLoggingSystemTests.java index 63a32cce5d..6152cfb3fd 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/LogbackLoggingSystemTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/logging/logback/LogbackLoggingSystemTests.java @@ -35,6 +35,7 @@ import ch.qos.logback.classic.LoggerContext; import ch.qos.logback.classic.spi.LoggerContextListener; import ch.qos.logback.core.ConsoleAppender; import ch.qos.logback.core.encoder.LayoutWrappingEncoder; +import ch.qos.logback.core.joran.spi.JoranException; import ch.qos.logback.core.rolling.RollingFileAppender; import ch.qos.logback.core.rolling.SizeAndTimeBasedRollingPolicy; import ch.qos.logback.core.util.DynamicClassLoadingException; @@ -847,8 +848,17 @@ class LogbackLoggingSystemTests extends AbstractLoggingSystemTests { assertThatIllegalStateException() .isThrownBy(() -> initialize(this.initializationContext, "classpath:logback-invalid-format.txt", getLogFile(tmpDir() + "/tmp.log", null))) - .satisfies((ex) -> assertThat(ex.getCause()).isInstanceOf(IllegalArgumentException.class) - .hasMessageStartingWith("Unsupported file extension")); + .satisfies((ex) -> assertThat(ex.getCause()).isInstanceOf(JoranException.class) + .hasMessageStartingWith("Problem parsing XML document. See previously reported errors")); + } + + @Test + void whenConfigLocationIsXmlFileWithoutExtensionShouldWork(CapturedOutput output) { + this.loggingSystem.beforeInitialize(); + initialize(this.initializationContext, "classpath:logback-without-extension", + getLogFile(tmpDir() + "/tmp.log", null)); + this.logger.info("No extension and works!"); + assertThat(output.toString()).contains("No extension and works!"); } @Test diff --git a/spring-boot-project/spring-boot/src/test/resources/logback-without-extension b/spring-boot-project/spring-boot/src/test/resources/logback-without-extension new file mode 100644 index 0000000000..68080b7c97 --- /dev/null +++ b/spring-boot-project/spring-boot/src/test/resources/logback-without-extension @@ -0,0 +1,11 @@ + + + + + %msg + + + + + +