[PR #4344] [MERGED] SSL for Streams - 2025 #3864

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

📋 Pull Request Information

Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/4344
Author: @jc21
Created: 2/5/2025
Status: Merged
Merged: 2/5/2025
Merged by: @jc21

Base: developHead: stream-ssl


📝 Commits (9)

  • 3091c21 Add SSL certificate to TCP streams if certificate in database
  • 3dbc70f Add SSL tab to stream UI
  • ee4250d Add SSL column to streams table UI
  • cd80cc8 Add certificate to streams database model
  • 4452f01 Fix whitespace in nginx stream config
  • 2657af9 Fix stream update not persisting
  • 68a7803 Fix api schema after merging latest changes
  • b4793d3 Adds testssl.sh and mkcert to cypress stack
  • 6a60627 Cypress test for Streams

📊 Changes

26 files changed (+818 additions, -182 deletions)

View changed files

📝 backend/internal/stream.js (+98 -21)
backend/migrations/20240427161436_stream_ssl.js (+38 -0)
📝 backend/models/stream.js (+18 -8)
📝 backend/schema/components/stream-object.json (+17 -3)
📝 backend/schema/paths/nginx/streams/get.json (+3 -2)
📝 backend/schema/paths/nginx/streams/post.json (+5 -1)
📝 backend/schema/paths/nginx/streams/streamID/get.json (+2 -1)
📝 backend/schema/paths/nginx/streams/streamID/put.json (+25 -65)
📝 backend/templates/_certificates.conf (+1 -0)
backend/templates/_certificates_stream.conf (+13 -0)
📝 backend/templates/stream.conf (+8 -12)
📝 docker/docker-compose.ci.yml (+5 -1)
docker/rootfs/etc/nginx/conf.d/include/ssl-cache-stream.conf (+2 -0)
docker/rootfs/etc/nginx/conf.d/include/ssl-cache.conf (+2 -0)
📝 docker/rootfs/etc/nginx/conf.d/include/ssl-ciphers.conf (+0 -3)
📝 docker/scripts/install-s6 (+1 -1)
📝 frontend/js/app/nginx/stream/form.ejs (+176 -37)
📝 frontend/js/app/nginx/stream/form.js (+154 -13)
📝 frontend/js/app/nginx/stream/list/item.ejs (+7 -1)
📝 frontend/js/app/nginx/stream/list/main.ejs (+1 -0)

...and 6 more files

📄 Description

Update upon #3789


🔄 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/4344 **Author:** [@jc21](https://github.com/jc21) **Created:** 2/5/2025 **Status:** ✅ Merged **Merged:** 2/5/2025 **Merged by:** [@jc21](https://github.com/jc21) **Base:** `develop` ← **Head:** `stream-ssl` --- ### 📝 Commits (9) - [`3091c21`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/3091c21caef3fc68fefb105b26ef7cb11e5ef1a9) Add SSL certificate to TCP streams if certificate in database - [`3dbc70f`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/3dbc70faa6619d3918a2528f9ac9f632116c2450) Add SSL tab to stream UI - [`ee4250d`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/ee4250d770f7b9792e201107ef96798d4543a0da) Add SSL column to streams table UI - [`cd80cc8`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/cd80cc8e4db0edcd482ce8fe99be2c21e589a473) Add certificate to streams database model - [`4452f01`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/4452f014b9b0125e2a6cf07d1f46e104355e912d) Fix whitespace in nginx stream config - [`2657af9`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/2657af97cf1e0ec46fdc4cde43e2a69482993d4c) Fix stream update not persisting - [`68a7803`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/68a78035134395cf9ef4a0739a9e6ccc9ee568cc) Fix api schema after merging latest changes - [`b4793d3`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/b4793d3c162561f7a4d13dc61947334eeea618d9) Adds testssl.sh and mkcert to cypress stack - [`6a60627`](https://github.com/NginxProxyManager/nginx-proxy-manager/commit/6a606278339f87cd6ed5769d375e706359a8c61c) Cypress test for Streams ### 📊 Changes **26 files changed** (+818 additions, -182 deletions) <details> <summary>View changed files</summary> 📝 `backend/internal/stream.js` (+98 -21) ➕ `backend/migrations/20240427161436_stream_ssl.js` (+38 -0) 📝 `backend/models/stream.js` (+18 -8) 📝 `backend/schema/components/stream-object.json` (+17 -3) 📝 `backend/schema/paths/nginx/streams/get.json` (+3 -2) 📝 `backend/schema/paths/nginx/streams/post.json` (+5 -1) 📝 `backend/schema/paths/nginx/streams/streamID/get.json` (+2 -1) 📝 `backend/schema/paths/nginx/streams/streamID/put.json` (+25 -65) 📝 `backend/templates/_certificates.conf` (+1 -0) ➕ `backend/templates/_certificates_stream.conf` (+13 -0) 📝 `backend/templates/stream.conf` (+8 -12) 📝 `docker/docker-compose.ci.yml` (+5 -1) ➕ `docker/rootfs/etc/nginx/conf.d/include/ssl-cache-stream.conf` (+2 -0) ➕ `docker/rootfs/etc/nginx/conf.d/include/ssl-cache.conf` (+2 -0) 📝 `docker/rootfs/etc/nginx/conf.d/include/ssl-ciphers.conf` (+0 -3) 📝 `docker/scripts/install-s6` (+1 -1) 📝 `frontend/js/app/nginx/stream/form.ejs` (+176 -37) 📝 `frontend/js/app/nginx/stream/form.js` (+154 -13) 📝 `frontend/js/app/nginx/stream/list/item.ejs` (+7 -1) 📝 `frontend/js/app/nginx/stream/list/main.ejs` (+1 -0) _...and 6 more files_ </details> ### 📄 Description Update upon #3789 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 08:32:14 +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#3864
No description provided.