Reject binding non-uniform property key as map key

Given:
```yaml
my:
  map:
    "/key": "value"
```
---
Before this commit:
It's equivalent to
```yaml
my:
  map:
    "[key]": "value" # "[/key]" is expected
```
Such counter-intuitive behavior will confuse developers, there are several reported issues, incomplete list: GH-41099 GH-29582 GH-24548

---
After this commit:
```
***************************
APPLICATION FAILED TO START
***************************

Description:

Failed to bind properties under 'my.map' to java.util.Map<java.lang.String, java.lang.String>:

    Reason: java.lang.IllegalArgumentException: Please rewrite key '/key' to '[/key]' and surround it with quotes if YAML is using

Action:

Update your application's configuration

```
---

See GH-42802

Author: quaff

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/bind/MapBinder.java

@@ -37,6 +37,7 @@
  *
  * @author Phillip Webb
  * @author Madhura Bhave
+ * @author Yanming Zhou
  */
 class MapBinder extends AggregateBinder<Map<Object, Object>> {
 
@@ -166,6 +167,17 @@ private class EntryBinder {
 		void bindEntries(ConfigurationPropertySource source, Map<Object, Object> map) {
 			if (source instanceof IterableConfigurationPropertySource iterableSource) {
 				for (ConfigurationPropertyName name : iterableSource) {
+					if (name.isLastElementNonUniform()) {
+						String sourceName = name.getSource();
+						int index = sourceName.lastIndexOf('.');
+						String key = sourceName.substring(index + 1);
+						if (!key.equals(name.getLastElement(Form.ORIGINAL))
+								&& !key.equalsIgnoreCase(name.getLastElement(Form.UNIFORM))) {
+							// ignore non-uniform caused by case-insensitive
+							throw new IllegalArgumentException("Please rewrite key '" + key + "' to '[" + key
+									+ "]' and surround it with quotes if YAML is using");
+						}
+					}
 					Bindable<?> valueBindable = getValueBindable(name);
 					ConfigurationPropertyName entryName = getEntryName(source, name);
 					Object key = getContext().getConverter().convert(getKeyName(entryName), this.keyType);

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/source/ConfigurationPropertyName.java

@@ -46,6 +46,7 @@
  *
  * @author Phillip Webb
  * @author Madhura Bhave
+ * @author Yanming Zhou
  * @since 2.0.0
  * @see #of(CharSequence)
  * @see ConfigurationPropertySource
@@ -121,6 +122,23 @@ public boolean isNumericIndex(int elementIndex) {
 		return this.elements.getType(elementIndex) == ElementType.NUMERICALLY_INDEXED;
 	}
 
+	/**
+	 * Return whether the last element is non-uniform.
+	 * @return whether the last element is non-uniform
+	 */
+	public boolean isLastElementNonUniform() {
+		int size = getNumberOfElements();
+		return size != 0 && this.elements.getType(size - 1) == ElementType.NON_UNIFORM;
+	}
+
+	/**
+	 * Return the source.
+	 * @return the source
+	 */
+	public String getSource() {
+		return this.elements.source.toString();
+	}
+
 	/**
 	 * Return the last element in the name in the given form.
 	 * @param form the form to return

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/context/properties/bind/MapBinderTests.java

@@ -64,6 +64,7 @@
  *
  * @author Phillip Webb
  * @author Madhura Bhave
+ * @author Yanming Zhou
  */
 class MapBinderTests {
 
@@ -651,6 +652,42 @@ void bindToEnumMapShouldBind() {
 		assertThat(result).hasSize(1).containsEntry(ExampleEnum.FOO_BAR, "value");
 	}
 
+	@Test
+	void rejectNonUniformKey() {
+		MapConfigurationPropertySource source = new MapConfigurationPropertySource(
+				Collections.singletonMap("props./foobar", "value"));
+		this.sources.add(source);
+		Binder binder = new Binder(this.sources, null, null, null);
+		Bindable<Map<String, String>> bindable = Bindable
+			.of(ResolvableType.forClassWithGenerics(Map.class, String.class, String.class));
+		assertThatExceptionOfType(BindException.class).isThrownBy(() -> binder.bind("props", bindable))
+			.withCauseInstanceOf(IllegalArgumentException.class);
+	}
+
+	@Test
+	void doNotRejectUnderscoreNonUniformKey() {
+		MapConfigurationPropertySource source = new MapConfigurationPropertySource(
+				Collections.singletonMap("props.foo_bar", "value"));
+		this.sources.add(source);
+		Binder binder = new Binder(this.sources, null, null, null);
+		Bindable<Map<String, String>> bindable = Bindable
+			.of(ResolvableType.forClassWithGenerics(Map.class, String.class, String.class));
+		Map<String, String> result = binder.bind("props", bindable).get();
+		assertThat(result).hasSize(1).containsEntry("foo_bar", "value");
+	}
+
+	@Test
+	void doNotRejectCaseInsensitiveNonUniformKey() {
+		MapConfigurationPropertySource source = new MapConfigurationPropertySource(
+				Collections.singletonMap("props.FOOBAR", "value"));
+		this.sources.add(source);
+		Binder binder = new Binder(this.sources, null, null, null);
+		Bindable<Map<String, String>> bindable = Bindable
+			.of(ResolvableType.forClassWithGenerics(Map.class, String.class, String.class));
+		Map<String, String> result = binder.bind("props", bindable).get();
+		assertThat(result).hasSize(1).containsEntry("FOOBAR", "value");
+	}
+
 	private <K, V> Bindable<Map<K, V>> getMapBindable(Class<K> keyGeneric, ResolvableType valueType) {
 		ResolvableType keyType = ResolvableType.forClass(keyGeneric);
 		return Bindable.of(ResolvableType.forClassWithGenerics(Map.class, keyType, valueType));