Removing need to supply principal/keytab/realm

- you should instead kinit before running your python application

Author: ricky

happybase/connection.py

@@ -86,8 +86,8 @@ class Connection(object):
 
     The optional `use_kerberos` argument allows you to establish a
     secure connection to HBase. This argument requires a buffered
-    `transport` protocol. You must provide a `realm`, `principal`
-    and `keytab` in order for authentication to work.
+    `transport` protocol. You must first authorize yourself with
+    your KDC by using kinit (e.g. kinit -kt my.keytab user@REALM)
 
     .. versionadded:: 0.9
        `protocol` argument
@@ -111,16 +111,12 @@ class Connection(object):
     :param str transport: Thrift transport mode (optional)
     :param bool use_kerberos: Connect to HBase via a secure connection (default: False)
     :param str sasl_service: The name of the SASL service (default: hbase)
-    :param str realm: Name of the kerberos realm (required if using sasl transport)
-    :param str principal: Kerberos principal to authenticate as (required if using sasl transport)
-    :param str keytab: Location of keytab file for authentication (required if using sasl transport)
     """
     def __init__(self, host=DEFAULT_HOST, port=DEFAULT_PORT, timeout=None,
                  autoconnect=True, table_prefix=None,
                  table_prefix_separator='_', compat=DEFAULT_COMPAT,
                  transport=DEFAULT_TRANSPORT, protocol=DEFAULT_PROTOCOL,
-                 use_kerberos=False, sasl_service="hbase", realm=None,
-                 principal=None, keytab=None):
+                 use_kerberos=False, sasl_service="hbase"):
 
         if transport not in THRIFT_TRANSPORTS:
             raise ValueError("'transport' must be one of %s"
@@ -151,9 +147,6 @@ def __init__(self, host=DEFAULT_HOST, port=DEFAULT_PORT, timeout=None,
         self.compat = compat
 
         self._use_kerberos = use_kerberos
-        self._keytab = keytab
-        self._principal = principal
-        self._realm = realm
         self._sasl_service = sasl_service
         self._transport_class = THRIFT_TRANSPORTS[transport]
         self._protocol_class = THRIFT_PROTOCOLS[protocol]
@@ -177,23 +170,10 @@ def _refresh_thrift_client(self):
             if self._transport_class is not TBufferedTransport:
                 raise ValueError("Must use a buffered transport "
                                  " when use_kerberos is enabled")
-            if not self._realm:
-                raise ValueError("A kerberos realm must be set"
-                                 " when use_kerberos is enabled")
-            if not self._principal:
-                raise ValueError("A kerberos principal must be set"
-                                 " when use_kerberos is enabled")
-            if not self._keytab:
-                raise ValueError("A kerberos keytab must be set"
-                                 " when use_kerberos is enabled")
-            if not path.isfile(self._keytab):
-                raise ValueError("Unable to open keytab [%s], make"
-                                 " sure that it exists" % self._keytab)
 
             saslc = sasl.Client()
             saslc.setAttr("host", self.host)
             saslc.setAttr("service", self._sasl_service)
-            saslc.setAttr("realm", self._realm)
             saslc.init()
             self.transport = TSaslClientTransport(saslc, "GSSAPI", socket)