[PR #1447] Investigate nginx-ui issue 1446 #5722

New issue

Closed

opened 2026-03-01 15:42:23 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 15:42:23 +03:00

Owner

Original Pull Request: https://github.com/0xJacky/nginx-ui/pull/1447

State: closed
Merged: Yes

Prevent disabled site health checks from making network requests to fix issue #1446.

Even when health checks were disabled, the tryGetFavicon function would implicitly trigger HTTP GET requests, leading to unexpected traffic. This PR ensures that when health checks are disabled, cached site metadata is returned immediately without any network activity.

Note

When health checks are disabled, return cached site metadata without making HTTP requests, leveraging a snapshot of prior site info and adding a unit test to enforce this.

Backend (sitecheck):

Behavior change: CheckSite now returns cached metadata when HealthCheckEnabled is false, avoiding favicon fetches and all network calls; populates fields from an existing site snapshot if available.

Utility: Adds getExistingSiteSnapshot with read-locking to safely clone prior SiteInfo.

Tests:

Adds TestCheckSiteSkipsNetworkWhenDisabled to ensure no HTTP requests occur when health checks are disabled.

^{Written by Cursor Bugbot for commit 54d276447e. This will update automatically on new commits. Configure here.}

**Original Pull Request:** https://github.com/0xJacky/nginx-ui/pull/1447 **State:** closed **Merged:** Yes --- Prevent disabled site health checks from making network requests to fix issue #1446. Even when health checks were disabled, the `tryGetFavicon` function would implicitly trigger HTTP GET requests, leading to unexpected traffic. This PR ensures that when health checks are disabled, cached site metadata is returned immediately without any network activity. --- <a href="https://cursor.com/background-agent?bcId=bc-577bb3c0-699c-4e68-b2ab-9685bf68e0b7"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-cursor-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-cursor-light.svg"><img alt="Open in Cursor" src="https://cursor.com/open-in-cursor.svg"></picture></a> <a href="https://cursor.com/agents?id=bc-577bb3c0-699c-4e68-b2ab-9685bf68e0b7"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/open-in-web-dark.svg"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/open-in-web-light.svg"><img alt="Open in Web" src="https://cursor.com/open-in-web.svg"></picture></a>  --- > [!NOTE] > When health checks are disabled, return cached site metadata without making HTTP requests, leveraging a snapshot of prior site info and adding a unit test to enforce this. > > - **Backend (sitecheck)**: > - **Behavior change**: `CheckSite` now returns cached metadata when `HealthCheckEnabled` is false, avoiding favicon fetches and all network calls; populates fields from an existing site snapshot if available. > - **Utility**: Adds `getExistingSiteSnapshot` with read-locking to safely clone prior `SiteInfo`. > - **Tests**: > - Adds `TestCheckSiteSkipsNetworkWhenDisabled` to ensure no HTTP requests occur when health checks are disabled. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 54d276447ed24ebebfd1425cf21ce653a49ecb9f. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup>