Merge pull request #7639 from chaodhib/patch-1

* pr/7639:
  Broaden LoggingApplicationListener ignores

Author: philwebb

spring-boot/src/main/java/org/springframework/boot/logging/LoggingApplicationListener.java

@@ -314,12 +314,7 @@ private void initializeSystem(ConfigurableEnvironment environment,
 	}
 
 	private boolean ignoreLogConfig(String logConfig) {
-		return !StringUtils.hasLength(logConfig)
-				|| isDefaultAzureLoggingConfig(logConfig);
-	}
-
-	private boolean isDefaultAzureLoggingConfig(String candidate) {
-		return candidate.startsWith("-Djava.util.logging.config.file=");
+		return !StringUtils.hasLength(logConfig) || logConfig.startsWith("-D");
 	}
 
 	private void initializeFinalLoggingLevels(ConfigurableEnvironment environment,

spring-boot/src/test/java/org/springframework/boot/logging/LoggingApplicationListenerTests.java

@@ -162,6 +162,18 @@ public void azureDefaultLoggingConfigDoesNotCauseAFailure() throws Exception {
 		assertThat(new File(tmpDir() + "/spring.log").exists()).isFalse();
 	}
 
+	@Test
+	public void tomcatNopLoggingConfigDoesNotCauseAFailure() throws Exception {
+		TestPropertySourceUtils.addInlinedPropertiesToEnvironment(this.context,
+				"logging.config: -Dnop");
+		this.initializer.initialize(this.context.getEnvironment(),
+				this.context.getClassLoader());
+		this.logger.info("Hello world");
+		String output = this.outputCapture.toString().trim();
+		assertThat(output).contains("Hello world").doesNotContain("???");
+		assertThat(new File(tmpDir() + "/spring.log").exists()).isFalse();
+	}
+
 	@Test
 	public void overrideConfigBroken() throws Exception {
 		TestPropertySourceUtils.addInlinedPropertiesToEnvironment(this.context,