mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[PR #451] [MERGED] Remove buffering from nginx config, make config secure by default #1305
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#1305
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/binwiederhier/ntfy/pull/451
Author: @SuperSandro2000
Created: 10/23/2022
Status: ✅ Merged
Merged: 11/2/2022
Merged by: @binwiederhier
Base:
main← Head:patch-1📝 Commits (1)
2d39ae1Remove buffering from nginx config, make config secure by default📊 Changes
1 file changed (+6 additions, -25 deletions)
View changed files
📝
docs/config.md(+6 -25)📄 Description
Turning off proxy buffering is not recommend by upstream https://www.nginx.com/blog/avoiding-top-10-nginx-configuration-mistakes/#proxy_buffering-off by default. And making configuration more secure by removing TLSv1 TLSv1.1 and redirecting to https all the time to never leak credentials.
PS: https is not annoying and curl can follow redirects with -L.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.