centralcli 7.2.1__tar.gz → 7.2.4__tar.gz

{centralcli-7.2.1 → centralcli-7.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: centralcli
-Version: 7.2.1
+Version: 7.2.4
 Summary: A CLI for interacting with Aruba Central (Cloud Management Platform).  Facilitates bulk imports, exports, reporting.  A handy tool if you have devices managed by Aruba Central.
 Home-page: https://github.com/Pack3tL0ss/central-api-cli
 License: MIT

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/__init__.py

@@ -194,15 +194,16 @@ if "--capture-raw" in sys.argv:  # captures raw responses into a flat file for l
 central = CentralApi(config.account)
 Cache.set_config(config)
 cache = Cache(central)
-CacheDevice.set_db(cache.DevDB)
-CacheInvDevice.set_db(cache.InvDB)
-CacheGroup.set_db(cache.GroupDB)
-CacheSite.set_db(cache.SiteDB)
-CacheClient.set_db(cache.ClientDB, cache=cache)
-CacheLabel.set_db(cache.LabelDB)
-CachePortal.set_db(cache.PortalDB)
-CacheTemplate.set_db(cache.TemplateDB)
-CacheMpskNetwork.set_db(cache.MpskDB)
+if config.valid:
+    CacheDevice.set_db(cache.DevDB)
+    CacheInvDevice.set_db(cache.InvDB)
+    CacheGroup.set_db(cache.GroupDB)
+    CacheSite.set_db(cache.SiteDB)
+    CacheClient.set_db(cache.ClientDB, cache=cache)
+    CacheLabel.set_db(cache.LabelDB)
+    CachePortal.set_db(cache.PortalDB)
+    CacheTemplate.set_db(cache.TemplateDB)
+    CacheMpskNetwork.set_db(cache.MpskDB)
 cli = CLICommon(config.account, cache, central, raw_out=raw_out)
 
 # allow singular form and common synonyms for the defined show commands

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/cache.py

@@ -464,6 +464,10 @@ class CachePortal(CentralObject):
     def __rich__(self) -> str:
         return f'[bright_green]Portal Profile[/]:[bright_green]{self.name}[/]|[cyan]{self.id}[/]'
 
+    @property
+    def help_text(self):
+        return render.rich_capture(self.__rich__())
+
 
 class CacheTemplate(CentralObject):
     db: Table | None = None
@@ -560,6 +564,12 @@ class CacheClient(CentralObject):
     def __rich__(self) -> str:
         return f'[bright_green]Client[/]:[cyan]{self.name}[/]|({utils.color([self.type, self.ip, self.mac, self.connected_name],  "green_yellow", sep="|")}|s[green_yellow]{self.site})[/]'
 
+    @property
+    def help_text(self) -> str:
+        return render.rich_capture(
+            f"[bright_green]{self.name}[/]|[cyan]{self.mac}[/]|[bright_green]{self.ip}[/]|[cyan]{f's:{self.site}' if self.site else f'g:{self.group}'}[/]|[dark_olive_green2]{self.connected_name}[/]"
+        )
+
 
 class CacheMpskNetwork(CentralObject):
     db: Table | None = None
@@ -1349,9 +1359,9 @@ class Cache:
             match = [m for m in match if m.name not in args]
             for m in sorted(match, key=lambda i: i.name):
                 if m.name.startswith(incomplete):
-                    out += [tuple([m.name, m.id])]
+                    out += [tuple([m.name, m.help_text])]
                 elif m.id.startswith(incomplete):
-                    out += [tuple([m.id, m.name])]
+                    out += [tuple([m.id, m.help_text])]
                 else:
                     out += [tuple([m.name, m.help_text])]  # failsafe, shouldn't hit
 
@@ -1959,19 +1969,18 @@ class Cache:
             # remove clients that are already on the command line
             match = [m for m in match if m.name not in args]
             for c in sorted(match, key=lambda i: i.name):
-                if c.name.startswith(incomplete):
-                    out += c.help_text
+                if c.name.lower().startswith(incomplete.lower()):
+                    out += [(c.name, c.help_text)]
                 elif c.mac.strip(":.-").lower().startswith(incomplete.strip(":.-")):
-                    out += c.help_text
+                    out += [(c.mac, c.help_text)]
                 elif c.ip.startswith(incomplete):
-                    out += c.help_text
+                    out += [(c.ip, c.help_text)]
                 else:
                     # failsafe, shouldn't hit
-                    out += (c.help_text[0], f'{c.help_text[1]} FailSafe Match')
+                    out += [(c.name, f'{c.help_text} FailSafe Match')]
 
-
-        for c in out:  # TODO completion behavior has changed.  This works-around issue bash doesn't complete past 00: and zsh treats each octet as a dev name when : is used.
-            yield c[0].replace(":", "-"), c[1]
+        for c in out:
+            yield c[0].replace(":", "-"), c[1]  # TODO completion behavior has changed.  This works-around issue bash doesn't complete past 00: and zsh treats each octet as a dev name when : is used.
 
     def event_log_completion(
         self,
@@ -3218,27 +3227,30 @@ class Cache:
 
 
             # no match found initiate cache update
-            if retry and not match and ((dev_type and not cache_updated) or self.responses.dev is None):
-                dev_type_sfx = "" if not dev_type else f" [grey42 italic](Device Type: {utils.unlistify(dev_type)})[/]"
-                econsole.print(f"[dark_orange3]:warning:[/]  [bright_red]No Match found[/] for [cyan]{query_str}[/]{dev_type_sfx}.")
-                if FUZZ:
-                    if dev_type:
-                        fuzz_match, fuzz_confidence = process.extract(query_str, [d["name"] for d in self.devices if d["type"] in dev_type], limit=1)[0]
-                    else:
-                        fuzz_match, fuzz_confidence = process.extract(query_str, [d["name"] for d in self.devices], limit=1)[0]
-                    confirm_str = render.rich_capture(f"Did you mean [green3]{fuzz_match}[/]?")
-                    if fuzz_confidence >= 70 and typer.confirm(confirm_str):
-                        match = self.DevDB.search(self.Q.name == fuzz_match)
-                if not match:
-                    kwargs = {"dev_db": True}
-                    if include_inventory:
-                        _word = " & Inventory "
-                        kwargs["inv_db"] = True
-                    else:
-                        _word = " "
-                    econsole.print(f":arrows_clockwise: Updating Device{_word}Cache.")
-                    self.check_fresh(refresh=True, dev_type=dev_type, **kwargs )
-                    cache_updated = True  # Need this for scenario when dev_type is the only thing refreshed, as that does not update self.responses.dev
+            if retry and not match and self.responses.dev is None:
+                if dev_type and cache_updated:
+                    ...  # self.responses.dev is not currently updated if dev_type provided, but cache update may have already occured in this session.
+                else:
+                    dev_type_sfx = "" if not dev_type else f" [grey42 italic](Device Type: {utils.unlistify(dev_type)})[/]"
+                    econsole.print(f"[dark_orange3]:warning:[/]  [bright_red]No Match found[/] for [cyan]{query_str}[/]{dev_type_sfx}.")
+                    if FUZZ:
+                        if dev_type:
+                            fuzz_match, fuzz_confidence = process.extract(query_str, [d["name"] for d in self.devices if d["type"] in dev_type], limit=1)[0]
+                        else:
+                            fuzz_match, fuzz_confidence = process.extract(query_str, [d["name"] for d in self.devices], limit=1)[0]
+                        confirm_str = render.rich_capture(f"Did you mean [green3]{fuzz_match}[/]?")
+                        if fuzz_confidence >= 70 and typer.confirm(confirm_str):
+                            match = self.DevDB.search(self.Q.name == fuzz_match)
+                    if not match:
+                        kwargs = {"dev_db": True}
+                        if include_inventory:
+                            _word = " & Inventory "
+                            kwargs["inv_db"] = True
+                        else:
+                            _word = " "
+                        econsole.print(f":arrows_clockwise: Updating Device{_word}Cache.")
+                        self.check_fresh(refresh=True, dev_type=dev_type, **kwargs )
+                        cache_updated = True  # Need this for scenario when dev_type is the only thing refreshed, as that does not update self.responses.dev
 
             if match:
                 match = [Model(dev) for dev in match]

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/cleaner.py

@@ -1626,7 +1626,7 @@ def show_interfaces(data: List[dict] | dict, verbosity: int = 0, dev_type: DevTy
         key_order = verbosity_keys[verbosity]
         # send all key/value pairs through formatters
         data = [
-            dict(short_value(k, d.get(k),) for k in key_order) for d in data
+            dict(short_value(k, d[k],) for k in key_order if k in d) for d in data
         ]
     else:
         key_order = [*key_order, *data[-1].keys()]

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/clicommon.py

@@ -383,8 +383,7 @@ class CLICommon:
         else:
             return default
 
-    @staticmethod
-    def write_file(outfile: Path, outdata: str) -> None:
+    def write_file(self, outfile: Path, outdata: str) -> None:
         """Output data to file
 
         Args:
@@ -407,19 +406,22 @@ class CLICommon:
 
             print(f"\n[cyan]Writing output to {outfile}... ", end="")
 
-            out_msg = None
-            try:
-                if isinstance(outdata, (dict, list)):
-                    outdata = json.dumps(outdata, indent=4)
-                outfile.write_text(outdata)  # typer.unstyle(outdata) also works
-            except Exception as e:
-                outfile.write_text(f"{outdata}")
-                out_msg = f"Error ({e.__class__.__name__}) occurred during attempt to output to file.  " \
-                    "Used simple string conversion"
+            if not outfile.parent.is_dir():
+                self.econsole.print(f"[red]Directory Not Found[/]\n[dark_orange3]:warning:[/]  Unable to write output to [cyan]{outfile.name}[/].\nDirectory [cyan]{str(outfile.parent.absolute())}[/] [red]does not exist[/].")
+            else:
+                out_msg = None
+                try:
+                    if isinstance(outdata, (dict, list)):
+                        outdata = json.dumps(outdata, indent=4)
+                    outfile.write_text(outdata)  # typer.unstyle(outdata) also works
+                except Exception as e:
+                    outfile.write_text(f"{outdata}")
+                    out_msg = f"Error ({e.__class__.__name__}) occurred during attempt to output to file.  " \
+                        "Used simple string conversion"
 
-            print("[italic green]Done")
-            if out_msg:
-                log.warning(out_msg, show=True)
+                print("[italic green]Done")
+                if out_msg:
+                    log.warning(out_msg, show=True)
 
     @staticmethod
     def exit(msg: str = None, code: int = 1, emoji: bool = True) -> None:

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/clikick.py

@@ -8,7 +8,6 @@ from rich import print
 
 
 # Detect if called from pypi installed package or via cloned github repo (development)
-# TODO should be able to do this in __init__
 try:
     from centralcli import cli
 except (ImportError, ModuleNotFoundError) as e:
@@ -19,16 +18,18 @@ except (ImportError, ModuleNotFoundError) as e:
     else:
         print(pkg_dir.parts)
         raise e
-from centralcli.constants import IdenMetaVars
 
-iden = IdenMetaVars()
+from centralcli.constants import iden_meta
+from .cache import CacheClient
+from .models import Clients
+
 app = typer.Typer()
 
 @app.command(short_help="Disconnect all WLAN clients from an AP optionally for a specific SSID",)
 def all(
     device: str = typer.Argument(
         ...,
-        metavar=iden.dev,
+        metavar=iden_meta.dev,
         help="The AP to disconnect clients from",
         autocompletion=cli.cache.dev_ap_completion,
         show_default=False,
@@ -46,7 +47,7 @@ def all(
     dev = cli.cache.get_dev_identifier(device)
     _ssid_msg = "" if not ssid else f" on SSID [cyan]{ssid}[/]"
     print(f'Kick [bright_red]ALL[/] users connected to [cyan]{dev.name}[/]{_ssid_msg}')
-    if yes or typer.confirm("\nproceed?", abort=True):
+    if cli.confirm(yes):
         resp = cli.central.request(
             cli.central.kick_users,
             dev.serial,
@@ -56,12 +57,10 @@ def all(
         cli.display_results(resp, tablefmt="action")
 
 
-# TODO rather than drop option have cache remove users with last_connected > 30 days
 @app.command(short_help="Disconnect a WLAN client",)
 def client(
-    client: str = typer.Argument(..., metavar=iden.client, autocompletion=cli.cache.client_completion, show_default=False),
+    client: str = typer.Argument(..., metavar=iden_meta.client, autocompletion=cli.cache.client_completion, show_default=False),
     refresh: bool = typer.Option(False, "--refresh", "-R", help="Cache is used to determine what AP the client is connected to, which could be [red]stale[/]. This forces a cache update."),
-    drop: bool = typer.Option(False, "--drop", "-D", help="(implies -R): Drop all users from existing cache, then refresh.  By default any user that has ever connected is retained in the cache."),
     yes: bool = cli.options.yes,
     debug: bool = cli.options.debug,
     default: bool = cli.options.default,
@@ -75,15 +74,31 @@ def client(
 
     The [cyan]-R[/] flag can be used to force a cache refresh prior to performing the disconnect.
     """
-    if refresh or drop:
-        resp = cli.central.request(cli.cache.refresh_client_db, "wireless", truncate=drop)
+    if refresh:
+        resp = cli.central.request(cli.cache.refresh_client_db, client_type="wireless")
         if not resp:
             cli.display_results(resp, exit_on_fail=True)
 
-    client = cli.cache.get_client_identifier(client, exit_on_fail=True)
+    client: CacheClient = cli.cache.get_client_identifier(client, exit_on_fail=True)
+    if not client.last_connected:
+        if refresh:
+            cli.exit(f"Client {client} is not connected.")
+        else:
+            client_resp = cli.central.request(cli.cache.refresh_client_db, mac=client.mac)
+            if not client_resp:
+                cli.econsole.print(f"client {client} is not online according to cache, Failure occured attempting to fetch client details from API.")
+                cli.display_results(client_resp, exit_on_fail=True)
+
+            _clients = [CacheClient(c) for c in Clients(client_resp.output)]
+            online_client =  [c for c in _clients if c.last_connected is not None]
+            if online_client:
+                client = online_client[-1]
+            else:
+                client = _clients[-1]
+                cli.exit(f"Client {client} is not online: Failure Stage: {client_resp.output[-1].get('failure_stage', '')}, Reason: {client_resp.output[-1].get('failure_reason', '')}")
 
     print(f'Kick client [cyan]{client.name}[/], currently connected to [cyan]{client.connected_name}[/]')
-    if yes or typer.confirm("\nProceed?", abort=True):
+    if cli.confirm(yes):
         resp = cli.central.request(
             cli.central.kick_users,
             client.connected_serial,

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/clishow.py

@@ -10,6 +10,8 @@ import os
 from datetime import datetime
 from typing import List, Iterable, Literal, Dict, Any, Tuple, TYPE_CHECKING
 from pathlib import Path
+import getpass
+from jinja2 import Template
 from rich import print
 from rich.console import Console
 
@@ -48,6 +50,7 @@ from centralcli.constants import (
 )
 from centralcli.cache import CentralObject
 from centralcli.objects import DateTime
+from .strings import cron_weekly
 
 if TYPE_CHECKING:
     from .cache import CacheSite, CacheGroup, CacheLabel, CacheDevice
@@ -163,7 +166,7 @@ def _build_device_caption(resp: Response, *, inventory: bool = False, dev_type:
 
     # Put together counts caption string
     if status:
-        _cnt_str = ", ".join([f'[{"bright_green" if status.lower() == "up" else "red"}]{t}[/]: [cyan]{status_by_type[t]["total"]}[/]' for t in status_by_type])
+        _cnt_str = ", ".join([f'[{"bright_green" if status.lower() == "up" else "red"}]{status.capitalize()} {t if t != "ap" else "APs"}[/]: [cyan]{status_by_type[t]["total"]}[/]' for t in status_by_type])
     elif inventory:
         _cnt_str = f"Total in inventory: [cyan]{len(resp.output)}[/], "
         _cnt_str = _cnt_str + ", ".join(
@@ -558,7 +561,7 @@ def aps(
         if up and down:
             ...  # They used both flags.  ignore
         elif up or down:
-            status = "Down" if down else "Up"
+            status = "down" if down else "up"
 
         show_devices(
             aps, dev_type="ap", include_inventory=with_inv, verbosity=verbose, outfile=outfile, update_cache=update_cache, group=group, site=site, label=label, status=status,
@@ -2009,6 +2012,7 @@ def wlans(
         "calculate_client_count": True,
     }
 
+    # TODO specifying WLAN name ... is ignored if verbose
     tablefmt = cli.get_format(do_json=do_json, do_yaml=do_yaml, do_csv=do_csv, do_table=do_table, default="rich")
     if group:  # Specifying the group implies verbose (same # of API calls either way.)
         resp = central.request(central.get_full_wlan_list, group)
@@ -2032,7 +2036,7 @@ def wlans(
     else:
         resp = central.request(central.get_wlans, **params)
         caption = None
-        if resp:
+        if resp and not name:
             caption = [f'[green]{len(resp.output)}[/] SSIDs,  [green]{sum([wlan.get("client_count", 0) for wlan in resp.output])}[/] Wireless Clients.']
             caption += ["Summary Output, Specify the group ([cyan]--group GROUP[/])",  "or use the verbose flag ([cyan]`-v`[/]) for additional details"]
         cli.display_results(resp, tablefmt=tablefmt, title=title, caption=caption, pager=pager, outfile=outfile, sort_by=sort_by, reverse=reverse, cleaner=cleaner.get_wlans)
@@ -2875,7 +2879,7 @@ def portals(
 # TODO add sort_by completion, portal completion
 @app.command()
 def guests(
-    portal: str = typer.Argument(..., help="portal name", show_default=False,),
+    portal: str = typer.Argument(..., help="portal name", autocompletion=cli.cache.portal_completion, show_default=False,),
     sort_by: str = cli.options.sort_by,
     reverse: bool = cli.options.reverse,
     do_json: bool = cli.options.do_json,
@@ -3017,6 +3021,42 @@ def version(
     """
     cli.version_callback()
 
+@app.command(hidden=os.name != "posix")
+def cron(
+    accounts: List[str] = typer.Argument(None,),
+) -> None:
+    """Show contents of cron file that can be used to automate token refresh weekly.
+
+    This will keep the tokens valid, even if cencli is not used.
+    """
+    if os.name != "posix":
+        cli.econsole.print("This command is currently only supported on Linux using cron.  It is possible to do the same via Windows Task Scheduler.  Showing Linux cron.weekly output for reference.")
+
+    user = getpass.getuser()
+    exec_path = sys.argv[0]
+    py_path = sys.executable
+
+    config_data = {
+        "user": user,
+        "py_path": py_path,
+        "exec_path": exec_path,
+        "accounts": "" if not accounts else " ".join(accounts)
+    }
+
+    template = Template(cron_weekly)
+    config_out = template.render(config_data)
+
+    cli.econsole.rule("/etc/cron.weekly/cencli file contents")
+    cli.console.print(config_out)
+    cli.econsole.rule()
+
+    cli.econsole.print(
+        "Place the above contents into a file: /etc/cron.weekly/cencli [grey42 italic](requires sudo)[/]\n"
+        f"Alternatively you can pipe the output directly [cyan]cencli show cron {'' if not accounts else ' '.join(accounts)} | sudo tee /etc/cron.weekly/cencli[/]"
+        "\nThen make it executable: [cyan]sudo chmod +x /etc/cron.weekly/cencli[/]"
+        "\n\n[cyan]cencli refresh token[/] [dark_olive_green2 italic]command will always update the tokens for the default workspace (that's the -d flag)[/]"
+    )
+
 
 def _get_cencli_config() -> None:
     try:

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/clitest.py

@@ -167,7 +167,7 @@ def method(
         resp = _check_bool_to_str(args, kwargs, resp_str=str(e))
 
     attrs = {
-        k: v for k, v in resp.__dict__.items() if k not in ["output", "raw"] and (log.DEBUG or not k.startswith("_"))
+        k: v for k, v in resp.__dict__.items() if k not in ["output", "raw", "data_key"] and (log.DEBUG or not k.startswith("_"))
     }
 
     req = (

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/cliupdate.py

@@ -3,29 +3,34 @@
 
 from pathlib import Path
 import sys
-from typing import List, Union
+from typing import TYPE_CHECKING, List, Union
 # from typing import List
 import typer
 from rich import print
 from rich.console import Console
+from rich.text import Text
 from jinja2 import FileSystemLoader, Environment
 import yaml
 
 # Detect if called from pypi installed package or via cloned github repo (development)
 try:
-    from centralcli import utils, cli, caas, cleaner, BatchRequest, log
+    from centralcli import utils, cli, cleaner, BatchRequest, log
 except (ImportError, ModuleNotFoundError) as e:
     pkg_dir = Path(__file__).absolute().parent
     if pkg_dir.name == "centralcli":
         sys.path.insert(0, str(pkg_dir.parent))
-        from centralcli import utils, cli, caas, cleaner, BatchRequest, log
+        from centralcli import utils, cli, cleaner, BatchRequest, log
     else:
         print(pkg_dir.parts)
         raise e
 
-from centralcli.constants import IdenMetaVars, DevTypes, GatewayRole, state_abbrev_to_pretty
-from centralcli import CentralObject
+from .constants import IdenMetaVars, DevTypes, GatewayRole, state_abbrev_to_pretty
+from . import render
 from .cache import CacheTemplate
+from .caas import CaasAPI
+
+if TYPE_CHECKING:
+    from .cache import CacheDevice, CacheGroup
 
 
 SPIN_TXT_AUTH = "Establishing Session with Aruba Central API Gateway..."
@@ -277,7 +282,6 @@ def generate_template(template_file: Union[Path, str], var_file: Union[Path, str
 @app.command("config")
 def config_(
     group_dev: str = cli.arguments.group_dev,
-    # TODO collect multi-line input as option to paste in config
     cli_file: Path = typer.Argument(..., help="File containing desired config/template in CLI format.", exists=True, autocompletion=lambda incomplete: tuple(), show_default=False,),
     var_file: Path = typer.Argument(None, help="File containing variables for j2 config template.", exists=True, autocompletion=lambda incomplete: tuple(), show_default=False,),
     do_gw: bool = typer.Option(None, "--gw", help="Update group level config for gateways."),
@@ -289,18 +293,12 @@ def config_(
 ) -> None:
     """Update group or device level config (ap or gw).
     """
-    group_dev: CentralObject = cli.cache.get_identifier(group_dev, qry_funcs=["group", "dev"], device_type=["ap", "gw"])
+    group_dev: CacheDevice | CacheGroup = cli.cache.get_identifier(group_dev, qry_funcs=["group", "dev"], device_type=["ap", "gw"])
     config_out = utils.generate_template(cli_file, var_file=var_file)
     cli_cmds = utils.validate_config(config_out)
 
-    # TODO render.py module with helper function to return styled rule/line
-    console = Console(record=True, emoji=False)
-    console.begin_capture()
-    console.rule("Configuration to be sent")
-    console.print("\n".join([f"[green]{line}[/green]" for line in cli_cmds]))
-    console.rule()
-    console.print(f"\nUpdating {'group' if group_dev.is_group else group_dev.generic_type.upper()} [cyan]{group_dev.name}[/]")
-    _msg = console.end_capture()
+    output = render.output(cli_cmds)
+    output = Text.from_ansi(output.tty)
 
     if group_dev.is_group:
         device = None
@@ -313,7 +311,7 @@ def config_(
         if device and device.generic_type != "gw":
             cli.exit(f"Invalid input: --gw option conflicts with {device.name} which is an {device.generic_type}")
         use_caas = True
-        caasapi = caas.CaasAPI(central=cli.central)  # XXX Burried import
+        caasapi = CaasAPI(central=cli.central)
         node_iden = group_dev.name if group_dev.is_group else group_dev.mac
     elif do_ap or (device and device.generic_type == "ap"):
         if device and device.generic_type != "ap":
@@ -321,7 +319,10 @@ def config_(
         use_caas = False
         node_iden = group_dev.name if group_dev.is_group else group_dev.serial
 
-    typer.echo(_msg)
+    cli.console.rule("Configuration to be sent")
+    cli.console.print(output, emoji=False)
+    cli.console.rule()
+    cli.console.print(f"\nUpdating {'group' if group_dev.is_group else group_dev.generic_type.upper()} [cyan]{group_dev.name}[/]")
     if cli.confirm(yes):
         if use_caas:
             resp = cli.central.request(caasapi.send_commands, node_iden, cli_cmds)
@@ -329,8 +330,7 @@ def config_(
         else:
             # FIXME this is OK for group level ap config , for AP this method is not valid
             if group_dev.is_dev:
-                print("Not Implemented yet for AP device level updates")
-                raise typer.Exit(1)
+                cli.exit("Not Implemented yet for AP device level updates")
             resp = cli.central.request(cli.central.replace_ap_config, node_iden, cli_cmds)
             cli.display_results(resp, tablefmt="action")
 

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/models.py

@@ -411,7 +411,7 @@ class Client(BaseModel):
     @classmethod
     def pretty_dt(cls, dt: datetime) -> DateTime:
         if dt is None:  # TODO all with potential for there not to be a value need this
-            return None
+            return DateTime(None, "timediff")  # resolves PydanticSerializationError when model_dump_json is called on client with None for last_connected (Failed)
 
         return DateTime(dt.timestamp(), "timediff")
 

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/response.py

@@ -383,7 +383,8 @@ class Response:
                 r = r.replace("message: ", "").replace(self.output["message"], f'[red italic]{self.output["message"]}[/]')
 
         r = r.replace("failed:", "[red]failed[/]:").replace("FAILED", "[red]FAILED[/red]").replace("failed_devices", "[red]failed_devices[/]").replace("INVALID", "[red]INVALID[/]")
-        r = r.replace("SUCCESS", "[bright_green]SUCCESS[/]").replace("Success", "[bright_green]Success[/]").replace("success", "[bright_green]success[/]").replace("succeeded_devices", "[bright_green]succeeded_devices[/]")
+        r = r.replace("SUCCESS", "[bright_green]SUCCESS[/]").replace("Success", "[bright_green]Success[/]").replace("Success[/]fully", "Successfully[/]")
+        r = r.replace("Success", "[bright_green]Success[/]").replace("success", "[bright_green]success[/]").replace("succeeded_devices", "[bright_green]succeeded_devices[/]")
         r = r.replace("invalid_device", "[red]invalid_device[/]").replace("blocked_device", "[red]blocked_device[/red]").replace("ATHENA_ERROR_DEVICE_ALREADY_EXIST", "[italic dark_orange3]Device already exists[/]")
 
         # sanitize sensitive data for demos

{centralcli-7.2.1 → centralcli-7.2.4}/centralcli/strings.py

@@ -589,3 +589,10 @@ class ImportExamples:
         if key not in self.__dict__.keys():
             log.error(f"An attempt was made to get {key} attr from ImportExamples which is not defined.")
             return f":warning: [bright_red]Error[/] no str defined for [cyan]ImportExamples.{key}[/]"
+
+cron_weekly = """#!/usr/bin/env bash
+
+/bin/su -c "{{py_path}} {{exec_path}} refresh token -d {{accounts}}" {{user}} &&
+    logger -t centralcli "Token Refreshed via cron" ||
+    logger -t centralcli "Token Refresh returned error"
+"""

{centralcli-7.2.1 → centralcli-7.2.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "centralcli"
-version = "7.2.1"
+version = "7.2.4"
 description = "A CLI for interacting with Aruba Central (Cloud Management Platform).  Facilitates bulk imports, exports, reporting.  A handy tool if you have devices managed by Aruba Central."
 license = "MIT"
 authors = ["Wade Wells (Pack3tL0ss) <wade@consolepi.org>"]