[PR #433] [MERGED] fix: http-to-https use $host and 308 #4040

New issue

Closed

opened 2026-02-28 11:58:56 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 11:58:56 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/0xJacky/nginx-ui/pull/433
Author: @wcbing
Created: 7/5/2024
Status: ✅ Merged
Merged: 7/6/2024
Merged by: @0xJacky

Base: dev ← Head: patch-1

📝 Commits (2)

bd66092 fix: http-to-https use $host and 301
ff7e3e7 fix: http-to-https use 301

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 template/block/http-to-https.conf (+2 -2)

📄 Description

使用 $server_name 在使用通配符域名等情况下无法跳转，建议使用 $host。
强制HTTPS我觉得应该是永久重定向比较好吧。

_{🔄 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/433 **Author:** [@wcbing](https://github.com/wcbing) **Created:** 7/5/2024 **Status:** ✅ Merged **Merged:** 7/6/2024 **Merged by:** [@0xJacky](https://github.com/0xJacky) **Base:** `dev` ← **Head:** `patch-1` --- ### 📝 Commits (2) - [`bd66092`](https://github.com/0xJacky/nginx-ui/commit/bd6609226b4cd717cad07743f61c36db91847eef) fix: http-to-https use $host and 301 - [`ff7e3e7`](https://github.com/0xJacky/nginx-ui/commit/ff7e3e79ca660a903c071765e4d907708a6e8871) fix: http-to-https use 301 ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `template/block/http-to-https.conf` (+2 -2) </details> ### 📄 Description 使用 $server_name 在使用通配符域名等情况下无法跳转，建议使用 $host。强制HTTPS我觉得应该是永久重定向比较好吧。 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>