Merge pull request #73 from britive/develop

v1.2.2

Author: twratl

CHANGELOG.md

@@ -2,6 +2,33 @@
 
 All changes to the package starting with v0.3.1 will be logged here.
 
+## v1.2.2 [2023-03-17]
+#### What's New
+* None
+
+#### Enhancements
+* None
+
+#### Bug Fixes
+* Fix bug with `logout` command when no active credentials were available
+* Expand `--silent/-s` flag to the following commands
+  * `api`
+  * `cache profiles`
+  * `checkin`
+  * `login`
+  * `logout`
+  * `ls [profiles|environments|applications|secrets]`
+  * `request [submit|withdraw]`
+  * `secret view`
+  * `user`
+* Fix bug when saving profile alias when the `PROFILE` is only 2 parts instead of 3
+
+#### Dependencies
+* None
+
+#### Other
+* None
+
 ## v1.2.1 [2023-03-14]
 #### What's New
 * None

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = pybritive
-version = 1.2.1
+version = 1.2.2
 author = Britive Inc.
 author_email = support@britive.com
 description = A pure Python CLI for Britive

src/pybritive/britive_cli.py

@@ -104,11 +104,26 @@ def _cleanup_credentials(self):
         self.credential_manager.delete()
 
     def logout(self):
+        # if dealing with a token there is no concept of logout
         if self.token:
             raise click.ClickException('Logout not available when using an API token.')
-        self.login()
-        self.b.delete(f'https://{Britive.parse_tenant(self.tenant_name)}/api/auth')
-        self._cleanup_credentials()
+
+        self.tenant_name = self.config.get_tenant()['name']
+        self.tenant_alias = self.config.alias
+        self.set_credential_manager()
+
+        # let's see if we have credentials for this tenant already
+        # if we do we need to invalidate them at the tenant and clean them up on the client side
+        # if we don't have valid credentials for the tenant then there is no need to logout
+        if self.credential_manager.has_valid_credentials():
+
+            self.b = Britive(
+                tenant=self.tenant_name,
+                token=self.credential_manager.get_token(),
+                query_features=False
+            )
+            self.b.delete(f'https://{Britive.parse_tenant(self.tenant_name)}/api/auth')
+            self._cleanup_credentials()
 
     def debug(self, data: object, ignore_silent: bool = False):
         if debug_enabled:
@@ -183,11 +198,11 @@ def user(self):
         output = f'{username} @ {self.tenant_name}'
         if alias != self.tenant_name:
             output += f' (alias: {alias})'
-        self.print(output)
+        self.print(output, ignore_silent=True)
 
     def list_secrets(self):
         self.login()
-        self.print(self.b.my_secrets.list())
+        self.print(self.b.my_secrets.list(), ignore_silent=True)
 
     def list_profiles(self, checked_out: bool = False):
         self.login()
@@ -219,7 +234,7 @@ def list_profiles(self, checked_out: bool = False):
         if self.output_format == 'list':
             self.output_format = 'list-profiles'
 
-        self.print(data)
+        self.print(data, ignore_silent=True)
 
         # and set it back
         if self.output_format == 'list-profiles':
@@ -242,7 +257,7 @@ def list_applications(self):
 
             }
             data.append(row)
-        self.print(data)
+        self.print(data, ignore_silent=True)
 
     def list_environments(self):
         self.login()
@@ -262,7 +277,7 @@ def list_environments(self):
                 'Type': env['app_type']
             }
             data.append(row)
-        self.print(data)
+        self.print(data, ignore_silent=True)
 
     def _set_available_profiles(self):
         if not self.available_profiles:
@@ -555,7 +570,7 @@ def viewsecret(self, path, blocktime, justification, maxpolltime):
             pass
 
         # and finally print the secret data
-        self.print(value)
+        self.print(value, ignore_silent=True)
 
     def downloadsecret(self, path, blocktime, justification, maxpolltime, file):
         self._validate_justification(justification)
@@ -578,7 +593,7 @@ def downloadsecret(self, path, blocktime, justification, maxpolltime, file):
 
         if file == '-':
             try:
-                self.print(content.decode('utf-8'))
+                self.print(content.decode('utf-8'), ignore_silent=True)
             except UnicodeDecodeError as e:
                 raise click.ClickException(
                     'Secret file contents cannot be decoded to utf-8. '
@@ -707,7 +722,7 @@ def api(self, method, parameters={}, query=None):
                 pass
 
         # output the response, optionally filtering based on provided jmespath query/search
-        self.print(jmespath.search(query, response) if query else response)
+        self.print(jmespath.search(query, response) if query else response, ignore_silent=True)
 
     # yes - this method exits in b.my_access as _get_profile_and_environment_ids_given_names
     # but we are doing additional business logic here to enhance the cli experience so there is

src/pybritive/commands/api.py

@@ -18,9 +18,9 @@
     )
 )
 @build_britive
-@britive_options(names='query,output_format,tenant,token,passphrase,federation_provider')
+@britive_options(names='query,output_format,tenant,token,silent,passphrase,federation_provider')
 @click_smart_api_method_argument  # need to gracefully handle older version of click
-def api(ctx, query, output_format, tenant, token, passphrase, federation_provider, method):
+def api(ctx, query, output_format, tenant, token, silent, passphrase, federation_provider, method):
     """Exposes the Britive Python SDK methods to the CLI.
 
     Documentation on each SDK method can be found inside the Python SDK itself and on Github

src/pybritive/commands/cache.py

@@ -11,8 +11,8 @@ def cache():
 
 @cache.command()
 @build_britive
-@britive_options(names='tenant,token,passphrase,federation_provider')
-def profiles(ctx, tenant, token, passphrase, federation_provider):
+@britive_options(names='tenant,token,silent,passphrase,federation_provider')
+def profiles(ctx, tenant, token, silent, passphrase, federation_provider):
     """Cache profiles locally to facilitate auto-completion of profile names on checkin/checkout."""
     ctx.obj.britive.cache_profiles()
 

src/pybritive/commands/checkin.py

@@ -6,9 +6,9 @@
 
 @click.command()
 @build_britive
-@britive_options(names='tenant,token,passphrase,federation_provider')
+@britive_options(names='tenant,token,silent,passphrase,federation_provider')
 @click_smart_profile_argument
-def checkin(ctx, tenant, token, passphrase, federation_provider, profile):
+def checkin(ctx, tenant, token, silent, passphrase, federation_provider, profile):
     """Checkin a profile.
 
     This command takes 1 required argument `PROFILE`. This should be a string representation of the profile

src/pybritive/commands/login.py

@@ -5,8 +5,8 @@
 
 @click.command()
 @build_britive
-@britive_options(names='tenant,token,passphrase,federation_provider')
-def login(ctx, tenant, token, passphrase, federation_provider):
+@britive_options(names='tenant,token,silent,passphrase,federation_provider')
+def login(ctx, tenant, token, silent, passphrase, federation_provider):
     """Perform an interactive login to obtain temporary credentials.
 
     This only applies when an API token has not been specified via `--token,-T` or via environment variable

src/pybritive/commands/logout.py

@@ -5,8 +5,8 @@
 
 @click.command()
 @build_britive
-@britive_options(names='tenant,passphrase')
-def logout(ctx, tenant, passphrase):
+@britive_options(names='tenant,silent,passphrase')
+def logout(ctx, tenant, silent, passphrase):
     """Logout of an interactive login session.
 
     This only applies when an API token has not been specified via `--token,-T` or via environment variable

src/pybritive/commands/ls.py

@@ -11,32 +11,32 @@ def ls():
 
 @ls.command()
 @build_britive
-@britive_options(names='format,tenant,token,passphrase,federation_provider')
-def applications(ctx, output_format, tenant, token, passphrase, federation_provider):
+@britive_options(names='format,tenant,token,silent,passphrase,federation_provider')
+def applications(ctx, output_format, tenant, token, silent, passphrase, federation_provider):
     """List applications for the currently authenticated identity."""
     ctx.obj.britive.list_applications()
 
 
 @ls.command()
 @build_britive
-@britive_options(names='format,tenant,token,passphrase,federation_provider')
-def environments(ctx, output_format, tenant, token, passphrase, federation_provider):
+@britive_options(names='format,tenant,token,silent,passphrase,federation_provider')
+def environments(ctx, output_format, tenant, token, silent, passphrase, federation_provider):
     """List environments for the currently authenticated identity."""
     ctx.obj.britive.list_environments()
 
 
 @ls.command()
 @build_britive
-@britive_options(names='checked_out,output_format,tenant,token,passphrase,federation_provider')
-def profiles(ctx, checked_out, output_format, tenant, token, passphrase, federation_provider):
+@britive_options(names='checked_out,output_format,tenant,token,silent,passphrase,federation_provider')
+def profiles(ctx, checked_out, output_format, tenant, token, silent, passphrase, federation_provider):
     """List profiles for the currently authenticated identity."""
     ctx.obj.britive.list_profiles(checked_out=checked_out)
 
 
 @ls.command()
 @build_britive
-@britive_options(names='format,tenant,token,passphrase, federation_provider')
-def secrets(ctx, output_format, tenant, token, passphrase, federation_provider):
+@britive_options(names='format,tenant,token,silent,passphrase,federation_provider')
+def secrets(ctx, output_format, tenant, token, silent, passphrase, federation_provider):
     """List secrets for the currently authenticated identity."""
     ctx.obj.britive.list_secrets()
 

src/pybritive/commands/request.py

@@ -12,9 +12,9 @@ def request():
 
 @request.command()
 @build_britive
-@britive_options(names='justification,tenant,token,passphrase,federation_provider')
+@britive_options(names='justification,tenant,token,silent,passphrase,federation_provider')
 @click_smart_profile_argument
-def submit(ctx, justification, tenant, token, passphrase, federation_provider, profile):
+def submit(ctx, justification, tenant, token, silent, passphrase, federation_provider, profile):
     """Submit a request to checkout a profile.
 
     Only applicable for profiles which require approval. This command will NOT block/wait until the request is
@@ -33,9 +33,9 @@ def submit(ctx, justification, tenant, token, passphrase, federation_provider, p
 
 @request.command()
 @build_britive
-@britive_options(names='tenant,token,passphrase,federation_provider')
+@britive_options(names='tenant,token,silent,passphrase,federation_provider')
 @click_smart_profile_argument
-def withdraw(ctx, tenant, token, passphrase, federation_provider, profile):
+def withdraw(ctx, tenant, token, silent, passphrase, federation_provider, profile):
     """Withdraw a request to checkout a profile.
 
     Only applicable for profiles which require approval.