[PR #2582] [MERGED] Use ssl_reject_handshake to reject requests to default https site #3587

Closed
opened 2026-02-26 08:31:11 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/2582
Author: @TheBeeZee
Created: 2/3/2023
Status: Merged
Merged: 3/7/2023
Merged by: @jc21

Base: developHead: ssl_reject_handshake


📝 Commits (1)

  • a7f0c3b Use ssl_reject_handshake to reject requests to default https site

📊 Changes

2 files changed (+1 additions, -18 deletions)

View changed files

📝 docker/rootfs/etc/nginx/conf.d/default.conf (+1 -3)
📝 docker/rootfs/etc/services.d/nginx/run (+0 -15)

📄 Description

Instead of creating a dummy certificate, we can return an SSL protocol error, which will generate a descriptive error message in the browser.


🔄 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/NginxProxyManager/nginx-proxy-manager/pull/2582 **Author:** [@TheBeeZee](https://github.com/TheBeeZee) **Created:** 2/3/2023 **Status:** ✅ Merged **Merged:** 3/7/2023 **Merged by:** [@jc21](https://github.com/jc21) **Base:** `develop` ← **Head:** `ssl_reject_handshake` --- ### 📝 Commits (1) - [`a7f0c3b`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/a7f0c3b730678ae4352ade2829d891a3ce3cd3bc) Use ssl_reject_handshake to reject requests to default https site ### 📊 Changes **2 files changed** (+1 additions, -18 deletions) <details> <summary>View changed files</summary> 📝 `docker/rootfs/etc/nginx/conf.d/default.conf` (+1 -3) 📝 `docker/rootfs/etc/services.d/nginx/run` (+0 -15) </details> ### 📄 Description Instead of creating a dummy certificate, we can return an SSL protocol error, which will generate a descriptive error message in the browser. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 08:31:11 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/nginx-proxy-manager-NginxProxyManager#3587
No description provided.