From 896994d42f9ccbd9cbc42d123cdc0c0df8ccc14d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gast=C3=B3n=20Fournier?= Date: Wed, 1 Feb 2023 17:30:35 +0100 Subject: [PATCH] fix: when app count is zero because it just started (#3029) ## About the changes This fixes response time metrics with app names when the app just starts and has zero which is falsy. We want to compare against undefined (which means the snapshot is not yet ready) --- .../admin/network/NetworkOverview/NetworkOverview.tsx | 6 +++++- src/lib/middleware/response-time-metrics.ts | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/frontend/src/component/admin/network/NetworkOverview/NetworkOverview.tsx b/frontend/src/component/admin/network/NetworkOverview/NetworkOverview.tsx index 96de9d76572..dde7c7dda4d 100644 --- a/frontend/src/component/admin/network/NetworkOverview/NetworkOverview.tsx +++ b/frontend/src/component/admin/network/NetworkOverview/NetworkOverview.tsx @@ -4,6 +4,8 @@ import { useInstanceMetrics } from 'hooks/api/getters/useInstanceMetrics/useInst import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender'; import { Alert, styled } from '@mui/material'; import { unknownify } from 'utils/unknownify'; +import logoIcon from 'assets/icons/logoBg.svg'; +import { formatAssetPath } from 'utils/formatPath'; const StyledMermaid = styled(Mermaid)(({ theme }) => ({ '#mermaid .node rect': { @@ -61,7 +63,9 @@ export const NetworkOverview = () => { graph TD subgraph _[ ] direction BT - Unleash(
Unleash) + Unleash(
Unleash) ${apps .map( ({ label, reqs, type }, i) => diff --git a/src/lib/middleware/response-time-metrics.ts b/src/lib/middleware/response-time-metrics.ts index 3cce701f2a6..f7a0edfe3c3 100644 --- a/src/lib/middleware/response-time-metrics.ts +++ b/src/lib/middleware/response-time-metrics.ts @@ -21,7 +21,7 @@ export function responseTimeMetrics( let appName; if ( flagResolver.isEnabled('responseTimeWithAppName') && - (instanceStatsService.getAppCountSnapshot('7d') || + (instanceStatsService.getAppCountSnapshot('7d') ?? appNameReportingThreshold) < appNameReportingThreshold ) { appName = req.headers['unleash-appname'] ?? req.query.appName;