[PR #4] [MERGED] Make it easy to change config file path #6

New issue

Closed

opened 2026-02-25 22:30:25 +03:00 by kerem · 0 comments

kerem commented

2026-02-25 22:30:25 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ergin/nginx-cloudflare-real-ip/pull/4
Author: @dciancu
Created: 8/25/2019
Status: ✅ Merged
Merged: 11/19/2019
Merged by: @ergin

Base: master ← Head: develop

📝 Commits (1)

1dffef2 Make it easy to change config file path

📊 Changes

1 file changed (+12 additions, -9 deletions)

View changed files

📝 cloudflare-sync-ips.sh (+12 -9)

📄 Description

Personally I would place this in /etc/nginx/snippets, as this is where snippets like this should reside.
With this change it makes it easy to change the file location path, without introducing breaking changes (without changing config file path).

_{🔄 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/ergin/nginx-cloudflare-real-ip/pull/4 **Author:** [@dciancu](https://github.com/dciancu) **Created:** 8/25/2019 **Status:** ✅ Merged **Merged:** 11/19/2019 **Merged by:** [@ergin](https://github.com/ergin) **Base:** `master` ← **Head:** `develop` --- ### 📝 Commits (1) - [`1dffef2`](https://github.com/ergin/nginx-cloudflare-real-ip/commit/1dffef219260d9d5e9b54ad08c58c215b66061b0) Make it easy to change config file path ### 📊 Changes **1 file changed** (+12 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `cloudflare-sync-ips.sh` (+12 -9) </details> ### 📄 Description Personally I would place this in `/etc/nginx/snippets`, as this is where snippets like this should reside. With this change it makes it easy to change the file location path, without introducing breaking changes (without changing config file path). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>