Fix Sonar Issues

garyrussell · garyrussell · commit 50624103fa88 · 2022-06-16T12:10:54.000-04:00
diff --git a/spring-rabbit-stream/src/main/java/org/springframework/rabbit/stream/listener/StreamListenerContainer.java b/spring-rabbit-stream/src/main/java/org/springframework/rabbit/stream/listener/StreamListenerContainer.java
@@ -16,6 +16,8 @@
 
 package org.springframework.rabbit.stream.listener;
 
+import java.util.Arrays;
+
 import org.aopalliance.aop.Advice;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
@@ -170,7 +172,7 @@ public boolean isAutoStartup() {
 	public void setAdviceChain(Advice... advices) {
 		Assert.notNull(advices, "'advices' cannot be null");
 		Assert.noNullElements(advices, "'advices' cannot have null elements");
-		this.adviceChain = advices;
+		this.adviceChain = Arrays.copyOf(adviceChain, adviceChain.length);
 	}
 
 	@Override
diff --git a/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/MessageAckListener.java b/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/MessageAckListener.java
@@ -23,6 +23,7 @@
  * A listener for message ack when using {@link AcknowledgeMode#AUTO}.
  *
  * @author Cao Weibo
+ * @author Gary Russell
  * @since 2.4.6
  */
 @FunctionalInterface
@@ -33,9 +34,7 @@ public interface MessageAckListener {
 	 * @param success Whether ack succeed.
 	 * @param deliveryTag The deliveryTag of ack.
 	 * @param cause The cause of failed ack.
-	 *
-	 * @throws Exception the exception during callback.
 	 */
-	void onComplete(boolean success, long deliveryTag, @Nullable Throwable cause) throws Exception;
+	void onComplete(boolean success, long deliveryTag, @Nullable Throwable cause);
 
 }
