Fix reply-to handling logic for AMQP 1.0

The `reply-to` property from request messages is parsed to the
`org.springframework.amqp.core.Address` object.
This one removes the first `/` leaving `routingKey`, based on the mentioned `reply-to`,
 with a `queues/` prefix.
The value in the `reply-to` is already encoded, so we don't need to go extra parsing logic
in the `RabbitAmqpTemplate.toAmqpMessage()` in regards
`com.rabbitmq.client.amqp.Message.MessageAddressBuilder`

* Fix `RabbitAmqpTemplate.toAmqpMessage()` to check for `queues/` prefix before going down to
the `com.rabbitmq.client.amqp.Message.MessageAddressBuilder` logic.
Instead, use the `queue` value as is in the  `com.rabbitmq.client.amqp.Message.to()` property
adding required `/` at the beginning of the value
* Revert `RabbitAmqpMessageListenerAdapter.sendResponse()` logic to `this.rabbitAmqpTemplate.send()`
without modifying `replyToRoutingKey` value
* Fix typos in the `Address` Javadocs
* Move `log4j-slf4j-impl` to common dependencies
* Fix `ObservationIntegrationTests` sorting spans by they create time to avoid race condition on assertions

Author: artembilan

build.gradle

@@ -199,6 +199,7 @@ configure(javaProjects) { subproject ->
 		testImplementation 'org.springframework:spring-test'
 
 		testRuntimeOnly 'org.apache.logging.log4j:log4j-jcl'
+		testRuntimeOnly 'org.apache.logging.log4j:log4j-slf4j-impl'
 
 		testImplementation 'org.junit.jupiter:junit-jupiter-api'
 		testImplementation 'org.junit.jupiter:junit-jupiter-params'
@@ -426,7 +427,6 @@ project('spring-rabbit-stream') {
 
 		testImplementation 'org.testcontainers:testcontainers-rabbitmq'
 		testImplementation 'org.testcontainers:testcontainers-junit-jupiter'
-		testImplementation 'org.apache.logging.log4j:log4j-slf4j-impl'
 		testImplementation 'org.springframework:spring-webflux'
 		testImplementation 'io.micrometer:micrometer-observation-test'
 		testImplementation 'io.micrometer:micrometer-tracing-bridge-brave'
@@ -451,7 +451,6 @@ project('spring-rabbitmq-client') {
 
 		testImplementation 'org.testcontainers:testcontainers-rabbitmq'
 		testImplementation 'org.testcontainers:testcontainers-junit-jupiter'
-		testImplementation 'org.apache.logging.log4j:log4j-slf4j-impl'
 	}
 }
 

spring-rabbit/src/test/java/org/springframework/amqp/rabbit/support/micrometer/ObservationIntegrationTests.java

@@ -16,6 +16,7 @@
 
 package org.springframework.amqp.rabbit.support.micrometer;
 
+import java.util.Comparator;
 import java.util.List;
 import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.TimeUnit;
@@ -50,7 +51,7 @@
  *
  * @since 3.0
  */
-@RabbitAvailable(queues = { "int.observation.testQ1", "int.observation.testQ2" })
+@RabbitAvailable(queues = {"int.observation.testQ1", "int.observation.testQ2"})
 public class ObservationIntegrationTests extends SampleTestRunner {
 
 	@Override
@@ -67,6 +68,7 @@ public SampleTestRunnerConsumer yourCode() {
 			}
 
 			List<FinishedSpan> finishedSpans = bb.getFinishedSpans();
+			finishedSpans.sort(Comparator.comparing(FinishedSpan::getStartTimestamp));
 			SpansAssert.assertThat(finishedSpans)
 					.haveSameTraceId()
 					.hasSize(4);

spring-rabbit/src/test/resources/log4j2-test.xml

@@ -6,9 +6,9 @@
 		</Console>
 	</Appenders>
 	<Loggers>
-		<Logger name="org.springframework.amqp.rabbit" level="info"/>
-		<Logger name="org.springframework.beans.factory" level="info"/>
-		<Root level="info">
+		<Logger name="org.springframework.amqp.rabbit" level="warn"/>
+		<Logger name="org.springframework.beans.factory" level="warn"/>
+		<Root level="warn">
 			<AppenderRef ref="STDOUT" />
 		</Root>
 	</Loggers>