[PR #1458] [MERGED] feat: allow full HTTP status codes for health checks #1294

New issue

Closed

opened 2026-02-26 12:11:07 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 12:11:07 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/0xJacky/nginx-ui/pull/1458
Author: @0xJacky
Created: 11/28/2025
Status: ✅ Merged
Merged: 11/28/2025
Merged by: @0xJacky

Base: dev ← Head: cursor/update-nginx-ui-to-vue-3-gpt-5.1-codex-high-3100

📝 Commits (4)

426f653 Add 401 Unauthorized to SiteHealthCheckModal status codes
69add12 Checkpoint before follow-up message
2afc580 Refactor status code selection to use grouped options
c4e1bee Fix: Correct apostrophe in "I'm a teapot" status

📊 Changes

1 file changed (+124 additions, -19 deletions)

View changed files

📝 app/src/views/dashboard/components/SiteHealthCheckModal.vue (+124 -19)

📄 Description

Add 401 Unauthorized to the health check's expected status codes dropdown.

This allows sites requiring authentication to correctly configure 401 Unauthorized as an expected status, preventing them from being mistakenly flagged as unhealthy.

Note

Expands the health check "Expected Status Codes" dropdown to include all grouped HTTP 1xx–5xx codes, replacing hardcoded options.

Frontend

app/src/views/dashboard/components/SiteHealthCheckModal.vue:

Add StatusCodeOption/StatusCodeGroup interfaces and createStatusOption helper; define grouped status lists for 1xx–5xx.

Replace hardcoded ASelectOptions with grouped options using ASelectOptGroup and v-for to render all HTTP codes.

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

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/0xJacky/nginx-ui/pull/1458 **Author:** [@0xJacky](https://github.com/0xJacky) **Created:** 11/28/2025 **Status:** ✅ Merged **Merged:** 11/28/2025 **Merged by:** [@0xJacky](https://github.com/0xJacky) **Base:** `dev` ← **Head:** `cursor/update-nginx-ui-to-vue-3-gpt-5.1-codex-high-3100` --- ### 📝 Commits (4) - [`426f653`](https://github.com/0xJacky/nginx-ui/commit/426f653a8dfde21f914d2bf7c385123c34bdf320) Add 401 Unauthorized to SiteHealthCheckModal status codes - [`69add12`](https://github.com/0xJacky/nginx-ui/commit/69add12a86cbba3d34b2d200129408be02d00f38) Checkpoint before follow-up message - [`2afc580`](https://github.com/0xJacky/nginx-ui/commit/2afc580a0f9eb84d8de2367c03d097202eb7f328) Refactor status code selection to use grouped options - [`c4e1bee`](https://github.com/0xJacky/nginx-ui/commit/c4e1bee26ab9176dcad5f7e32572f39f6207ad99) Fix: Correct apostrophe in "I'm a teapot" status ### 📊 Changes **1 file changed** (+124 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `app/src/views/dashboard/components/SiteHealthCheckModal.vue` (+124 -19) </details> ### 📄 Description Add `401 Unauthorized` to the health check's expected status codes dropdown. This allows sites requiring authentication to correctly configure `401 Unauthorized` as an expected status, preventing them from being mistakenly flagged as unhealthy. --- <a href="https://cursor.com/background-agent?bcId=bc-399947a5-c208-487d-a0fb-ec1bb7397df2"><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-399947a5-c208-487d-a0fb-ec1bb7397df2"><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] > Expands the health check "Expected Status Codes" dropdown to include all grouped HTTP 1xx–5xx codes, replacing hardcoded options. > > - **Frontend** > - **`app/src/views/dashboard/components/SiteHealthCheckModal.vue`**: > - Add `StatusCodeOption`/`StatusCodeGroup` interfaces and `createStatusOption` helper; define grouped status lists for 1xx–5xx. > - Replace hardcoded `ASelectOption`s with grouped options using `ASelectOptGroup` and `v-for` to render all HTTP codes. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit c4e1bee26ab9176dcad5f7e32572f39f6207ad99. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup>  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>