Continue with default location if CONFIG_LOGGING fails

Tomcat in particular sets an environment variable in it's shell scripts
that people commonly use to start the container. So if people deploy a war
file to a stock Tomcat server they can't override the logging config, even
with the default location. With this change at least that should work
(for logback and log4j anyway). Tested with logback.

See gh-1432

Author: Dave Syer

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

@@ -185,19 +185,20 @@ protected void initialize(ConfigurableEnvironment environment, ClassLoader class
 				system.initialize(value);
 			}
 			catch (Exception ex) {
-				this.logger.warn("Logging environment value '" + value
-						+ "' cannot be opened and will be ignored");
+				this.logger
+						.warn("Logging environment value '"
+								+ value
+								+ "' cannot be opened and will be ignored (using default location instead)");
+				system.initialize();
 			}
 		}
 		else {
-
 			system.initialize();
-			if (this.springBootLogging != null) {
-				initializeLogLevel(system, this.springBootLogging);
-			}
-
 		}
 
+		if (this.springBootLogging != null) {
+			initializeLogLevel(system, this.springBootLogging);
+		}
 		setLogLevels(system, environment);
 
 	}

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

@@ -126,6 +126,11 @@ public void overrideConfigDoesNotExist() throws Exception {
 		this.initializer.initialize(this.context.getEnvironment(),
 				this.context.getClassLoader());
 		// Should not throw
+		this.logger.info("Hello world");
+		String output = this.outputCapture.toString().trim();
+		assertTrue("Wrong output:\n" + output, output.contains("Hello world"));
+		assertFalse("Wrong output:\n" + output, output.contains("???"));
+		assertTrue(new File(tmpDir() + "/spring.log").exists());
 	}
 
 	@Test