mirror of
https://github.com/NginxProxyManager/nginx-proxy-manager.git
synced 2026-04-25 09:25:55 +03:00
[PR #1182] [MERGED] Update ssl-ciphers.conf #3392
Labels
No labels
awaiting feedback
bug
cannot reproduce
dns provider request
duplicate
enhancement
enhancement
enhancement
good first issue
help wanted
invalid
need more info
no certbot plugin available
product-support
pull-request
question
stale
troll
upstream issue
v2
v2
v2
v3
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/nginx-proxy-manager-NginxProxyManager#3392
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/NginxProxyManager/nginx-proxy-manager/pull/1182
Author: @DSorlov
Created: 6/17/2021
Status: ✅ Merged
Merged: 6/19/2021
Merged by: @jc21
Base:
develop← Head:develop📝 Commits (1)
3e744b6Update ssl-ciphers.conf📊 Changes
1 file changed (+2 additions, -4 deletions)
View changed files
📝
docker/rootfs/etc/nginx/conf.d/include/ssl-ciphers.conf(+2 -4)📄 Description
Removing support (by default) for all the unsecure protocols for SSL. As it is now its very permissive and maximizing on backwards-compability causing bad results in security reviews such as ipv6.cool tests. The default value is revised in this PR to create a balance between security and backwards combability.
This is updated using the latest recommendation as found on https://ssl-config.mozilla.org/#server=nginx&version=1.17.7&config=intermediate&openssl=1.1.1d&guideline=5.6
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.