From 8ddb8c810a1ab3175954a34b1210bdef3c01d2a9 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 96de9d7657..dde7c7dda4 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 3cce701f2a..f7a0edfe3c 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;