You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by ee...@apache.org on 2010/03/11 17:26:13 UTC
svn commit: r921920 -
/incubator/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/cli/CliMain.java
Author: eevans
Date: Thu Mar 11 16:26:13 2010
New Revision: 921920
URL: http://svn.apache.org/viewvc?rev=921920&view=rev
Log:
cassandra-cli: login() should be optional until it's not
Patch by eevans
Modified:
incubator/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/cli/CliMain.java
Modified: incubator/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/cli/CliMain.java
URL: http://svn.apache.org/viewvc/incubator/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/cli/CliMain.java?rev=921920&r1=921919&r2=921920&view=diff
==============================================================================
--- incubator/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/cli/CliMain.java (original)
+++ incubator/cassandra/branches/cassandra-0.6/src/java/org/apache/cassandra/cli/CliMain.java Thu Mar 11 16:26:13 2010
@@ -97,33 +97,36 @@ public class CliMain
thriftClient_ = cassandraClient;
cliClient_ = new CliClient(css_, thriftClient_);
- // Authenticate
- Map<String, String> credentials = new HashMap<String, String>();
- credentials.put(SimpleAuthenticator.USERNAME_KEY, css_.username);
- credentials.put(SimpleAuthenticator.PASSWORD_KEY, css_.password);
- AuthenticationRequest authRequest = new AuthenticationRequest(credentials);
- try
- {
- thriftClient_.login(css_.keyspace, authRequest);
- }
- catch (AuthenticationException e)
+ if ((css_.username != null) && (css_.password != null) && (css_.keyspace != null))
{
- css_.err.println("Exception during authentication to the cassandra node, " +
- "verify you are using correct credentials.");
- return;
- }
- catch (AuthorizationException e)
- {
- css_.err.println("You are not authorized to use keyspace: " + css_.keyspace);
- return;
- }
- catch (TException e)
- {
- if (css_.debug)
- e.printStackTrace();
-
- css_.err.println("Login failure. Did you specify 'keyspace', 'username' and 'password'?");
- return;
+ // Authenticate
+ Map<String, String> credentials = new HashMap<String, String>();
+ credentials.put(SimpleAuthenticator.USERNAME_KEY, css_.username);
+ credentials.put(SimpleAuthenticator.PASSWORD_KEY, css_.password);
+ AuthenticationRequest authRequest = new AuthenticationRequest(credentials);
+ try
+ {
+ thriftClient_.login(css_.keyspace, authRequest);
+ }
+ catch (AuthenticationException e)
+ {
+ css_.err.println("Exception during authentication to the cassandra node, " +
+ "verify you are using correct credentials.");
+ return;
+ }
+ catch (AuthorizationException e)
+ {
+ css_.err.println("You are not authorized to use keyspace: " + css_.keyspace);
+ return;
+ }
+ catch (TException e)
+ {
+ if (css_.debug)
+ e.printStackTrace();
+
+ css_.err.println("Login failure. Did you specify 'keyspace', 'username' and 'password'?");
+ return;
+ }
}
// Lookup the cluster name, this is to make it clear which cluster the user is connected to