2800: Introduce config.flag_is_true() to Python SDK.

diff --git a/sdk/python/arvados/api.py b/sdk/python/arvados/api.py
index 3df24306df750bc0a4e574218cc86024cfe5b993..e7348a1356077e1ea8a670c59a071787b8414e4e 100644 (file)
--- a/sdk/python/arvados/api.py
+++ b/sdk/python/arvados/api.py
@@ -119,8 +119,7 @@ def api(version=None, cache=True, host=None, token=None, insecure=False, **kwarg
                 raise ValueError("%s is not set. Aborting." % x)
         host = config.get('ARVADOS_API_HOST')
         token = config.get('ARVADOS_API_TOKEN')
-        insecure = (config.get('ARVADOS_API_HOST_INSECURE', '').lower() in
-                       ('yes', 'true', '1'))
+        insecure = config.flag_is_true('ARVADOS_API_HOST_INSECURE')
     else:
         # Caller provided one but not the other
         if not host:

diff --git a/sdk/python/arvados/config.py b/sdk/python/arvados/config.py
index e205e9239a65818f7ce21863ed676d96a46c737c..2b8374afb5c493cbf0eaeecc4d95da5325433865 100644 (file)
--- a/sdk/python/arvados/config.py
+++ b/sdk/python/arvados/config.py
@@ -25,6 +25,9 @@ def initialize(config_file=default_config_file):
         if var.startswith('ARVADOS_'):
             _settings[var] = os.environ[var]
 
+def flag_is_true(key):
+    return get(key, '').lower() in set(['1', 't', 'true', 'y', 'yes'])
+
 def get(key, default_val=None):
     return settings().get(key, default_val)