Improve SpringEnvironmentLookup exception to hint that you might need a file ending '-spring' about spring-boot

[philwebb]

Closing in favor of PR #42405. Thanks @nosan!

[nosan]

@philwebb
The exception message could be something like this

	public String lookup(String key) {
		Assert.state(this.environment != null, """
				Unable to obtain Spring Environment from LoggerContext.
				Usually, it happens when log4j2.xml file is being used instead of log4j2-spring.xml
				""");
		return this.environment.getProperty(key);
	}

What do you think?

[philwebb]

@nosan I wonder if we should make it clear that other extensions might also be used. The person reporting the issue was using yaml I think!

Perhaps something like:

Unable to obtain Spring Environment from LoggerContext. This can happen if your logback configuration filename does not end '-spring' (for example using 'log4j2.xml' instead of 'log4j2-spring.xml')

We should probably also keep the message on one line I think.

[nosan]

public String lookup(String key) {
		Assert.state(this.environment != null,
				"Unable to obtain Spring Environment from LoggerContext. "
						+ "This can happen if your log4j2 configuration filename does not end with '-spring' "
						+ "(for example using 'log4j2.xml' instead of 'log4j2-spring.xml')");
		return this.environment.getProperty(key);
	}
	

The error message will be

Unable to obtain Spring Environment from LoggerContext. This can happen if your log4j2 configuration filename does not end with '-spring' (for example using 'log4j2.xml' instead of 'log4j2-spring.xml')

If it is ok, I can create PR 😄

[philwebb]

If it is ok, I can create PR 😄

ABSOLUTELY!! 👍