From 9253be892de5f60819e4fd562ff424da0623d019 Mon Sep 17 00:00:00 2001 From: Iain Learmonth Date: Sun, 26 Feb 2023 13:45:57 +0000 Subject: [PATCH] fix(portal): use hashed fingerprint and cloud account in logging --- app/terraform/block/bridge.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/app/terraform/block/bridge.py b/app/terraform/block/bridge.py index ca9bb24..d4162fc 100644 --- a/app/terraform/block/bridge.py +++ b/app/terraform/block/bridge.py @@ -31,13 +31,13 @@ class BlockBridgeAutomation(BaseAutomation): bridge = bridge_select_func(id_) if bridge is None: continue - logging.debug("Found %s blocked", bridge.fingerprint) + logging.debug("Found %s blocked", bridge.hashed_fingerprint) if bridge.added > datetime.utcnow() - timedelta(hours=3): logging.debug("Not rotating a bridge less than 3 hours old") continue if bridge.deprecate(reason=self.short_name): - logging.info("Rotated %s", bridge.fingerprint) - rotated.append((bridge.fingerprint, bridge.conf.provider)) + logging.info("Rotated %s", bridge.hashed_fingerprint) + rotated.append((bridge.fingerprint, bridge.cloud_account.provider, bridge.cloud_account.description)) else: logging.debug("Not rotating a bridge that is already deprecated") return rotated @@ -58,7 +58,7 @@ class BlockBridgeAutomation(BaseAutomation): activity = Activity( activity_type="block", text=(f"[{self.short_name}] ♻ Rotated {len(rotated)} bridges: \n" - + "\n".join([f"* {fingerprint} ({provider})" for fingerprint, provider in rotated])) + + "\n".join([f"* {fingerprint} ({provider}: {provider_description})" for fingerprint, provider, provider_description in rotated])) ) db.session.add(activity) activity.notify()