You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by am...@apache.org on 2005/07/13 23:22:12 UTC

svn commit: r216268 - /geronimo/trunk/modules/deploy-tool/src/java/org/apache/geronimo/deployment/cli/ServerConnection.java

Author: ammulder
Date: Wed Jul 13 14:22:11 2005
New Revision: 216268

URL: http://svn.apache.org/viewcvs?rev=216268&view=rev
Log:
Nicer message for login failure (GERONIMO-748)
Add a blank line after password prompt

Modified:
    geronimo/trunk/modules/deploy-tool/src/java/org/apache/geronimo/deployment/cli/ServerConnection.java

Modified: geronimo/trunk/modules/deploy-tool/src/java/org/apache/geronimo/deployment/cli/ServerConnection.java
URL: http://svn.apache.org/viewcvs/geronimo/trunk/modules/deploy-tool/src/java/org/apache/geronimo/deployment/cli/ServerConnection.java?rev=216268&r1=216267&r2=216268&view=diff
==============================================================================
--- geronimo/trunk/modules/deploy-tool/src/java/org/apache/geronimo/deployment/cli/ServerConnection.java (original)
+++ geronimo/trunk/modules/deploy-tool/src/java/org/apache/geronimo/deployment/cli/ServerConnection.java Wed Jul 13 14:22:11 2005
@@ -181,7 +181,7 @@
             if(authPrompt && (user == null || password == null)) {
                 doAuthPromptAndRetry(uri, commandContext, user, password);
             } else {
-                throw new DeploymentException("Unable to connect to server", e);
+                throw new DeploymentException("Login Failed");
             }
         } catch(DeploymentManagerCreationException e) {
             if(uri != null) {
@@ -320,7 +320,7 @@
                 }
                 out.print(clearline.toString());
                 out.flush();
-                out.print('\r');
+                out.println();
                 out.flush();
             } catch (InterruptedException e) {
             } finally {