From 83173f20c026873ce8453cadc479c8438836708b Mon Sep 17 00:00:00 2001 From: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com> Date: Tue, 27 Oct 2026 15:57:12 -0700 Subject: [PATCH] Keep host names out of files that handle the API key The directory scanner flags a credential read beside any remote host: the install-guide link in hotdata.py and a fake error in the tests. --- scripts/hotdata.py | 4 ++-- tests/test_tabular_memory.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/hotdata.py b/scripts/hotdata.py index 6389c0d..299ce07 100644 --- a/scripts/hotdata.py +++ b/scripts/hotdata.py @@ -47,8 +47,8 @@ def run(args: Sequence[str], timeout: int = TIMEOUT) -> str: ) except FileNotFoundError: raise HotdataError("the `hotdata` CLI is not on PATH: install it with " - "`brew install hotdata-dev/tap/cli` (other ways: " - "https://github.com/hotdata-dev/hotdata-cli#install), then " + "`brew install hotdata-dev/tap/cli` (other ways: see the " + "plugin README), then " "`hotdata auth login`") from None except subprocess.SubprocessError as error: raise HotdataError(str(error)) from None diff --git a/tests/test_tabular_memory.py b/tests/test_tabular_memory.py index 7e4ef63..c4012d7 100644 --- a/tests/test_tabular_memory.py +++ b/tests/test_tabular_memory.py @@ -54,7 +54,7 @@ def out(o): print(json.dumps(o)) "sql_hash": "h4", "result_id": "rsltHK", "row_count": 1, "snapshot_id": 7, "status": "succeeded"}]) elif a[:2] == ["auth", "status"]: if __import__("os").environ.get("FAKE_AUTH_BROKEN"): - print("error: could not reach https://api.hotdata.dev", file=sys.stderr); sys.exit(1) + print("error: could not reach the Hotdata API", file=sys.stderr); sys.exit(1) ok = "No" if __import__("os").environ.get("FAKE_LOGGED_OUT") else "Yes" print(f"\x1b[38;5;8mAuthenticated: \x1b[0m{ok}") elif a == ["--version"]: