Browse Source

Disallow setting `upstream-base-url` to the same value as `base-url`

Philipp Heckel 3 years ago
parent
commit
30c2a67869
2 changed files with 3 additions and 1 deletions
  1. 2 0
      cmd/serve.go
  2. 1 1
      docs/releases.md

+ 2 - 0
cmd/serve.go

@@ -162,6 +162,8 @@ func execServe(c *cli.Context) error {
 		return errors.New("if set, upstream-base-url must start with http:// or https://")
 	} else if upstreamBaseURL != "" && baseURL == "" {
 		return errors.New("if upstream-base-url is set, base-url must also be set")
+	} else if upstreamBaseURL != "" && baseURL != "" && baseURL == upstreamBaseURL {
+		return errors.New("base-url and upstream-base-url cannot be identical, you'll likely want to set upstream-base-url to https://ntfy.sh, see https://ntfy.sh/docs/config/#ios-instant-notifications")
 	}
 
 	webRootIsApp := webRoot == "app"

+ 1 - 1
docs/releases.md

@@ -13,7 +13,7 @@ and the [ntfy Android app](https://github.com/binwiederhier/ntfy-android/release
 **Bugs:**
 
 * Return HTTP 500 for GET /_matrix/push/v1/notify when base-url is not configured (no ticket)
-
+* Disallow setting `upstream-base-url` to the same value as `base-url` ([#334](https://github.com/binwiederhier/ntfy/issues/334), thanks to [@oester](https://github.com/oester) for reporting)
 
 ## ntfy Android app v1.14.0 (UNRELEASED)