From 38fc74749b6eb5f5e02f21ff2193073cb682bb28 Mon Sep 17 00:00:00 2001
From: Justin Bertram <jbertram@apache.org>
Date: Tue, 1 Nov 2022 10:22:55 -0500
Subject: [PATCH] ARTEMIS-4079 quick semantic fix

---
 .../artemis/cli/commands/messages/ConnectionAbstract.java   | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/messages/ConnectionAbstract.java b/artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/messages/ConnectionAbstract.java
index c9297b973e8..3d5911a3033 100644
--- a/artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/messages/ConnectionAbstract.java
+++ b/artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/messages/ConnectionAbstract.java
@@ -226,14 +226,16 @@ private String inputBrokerURL(String defaultValue) {
    private String inputUser(String user) {
       if (user == null) {
          this.user = input("--user", "Type the username for a retry", null);
+         return this.user;
       }
-      return this.user;
+      return user;
    }
 
    private String inputPassword(String password) {
       if (password == null) {
          this.password = inputPassword("--password", "Type the password for a retry", null);
+         return this.password;
       }
-      return this.password;
+      return password;
    }
 }