[GH-ISSUE #322] setting.yaml base-url comment might want to refer to iOS notifications #252

New issue

Closed

opened 2026-05-07 00:22:18 +02:00 by BreizhHardware · 3 comments

BreizhHardware commented

2026-05-07 00:22:18 +02:00

Owner

Originally created by @milksteakjellybeans on GitHub (Jun 11, 2022).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/322

When using the "upstream-base-url" option without using the "base-url" option ntfy serve will output:

if upstream-base-url is set, base-url must also be set

From settings.yaml:

# Public facing base URL of the service (e.g. https://ntfy.sh or https://ntfy.example.com)
# This setting is currently only used by the attachments and e-mail sending feature (outgoing mail only).
#
# base-url:

This^ section may need updating to refer to base-url use as part of the iOS notifications feature, in addition to the mention of attachments and email.

thank you

Originally created by @milksteakjellybeans on GitHub (Jun 11, 2022). Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/322 When using the "upstream-base-url" option without using the "base-url" option ntfy serve will output: ``` if upstream-base-url is set, base-url must also be set ``` From settings.yaml: ``` # Public facing base URL of the service (e.g. https://ntfy.sh or https://ntfy.example.com) # This setting is currently only used by the attachments and e-mail sending feature (outgoing mail only). # # base-url: ``` This^ section may need updating to refer to base-url use as part of the iOS notifications feature, in addition to the mention of attachments and email. thank you

BreizhHardware closed this issue

2026-05-07 00:22:19 +02:00

BreizhHardware commented

2026-05-07 00:22:19 +02:00

Author

Owner

@binwiederhier commented on GitHub (Jun 12, 2022):

Fixed in github.com/binwiederhier/ntfy@41096ef1b0

If you can, why don't you try to create a pull request for things like this (docs updates, simple changes) next time. That'd help me a lot :-) If not, don't worry about it.

Thank you for reporting this (and the other tickets!!)

@binwiederhier commented on GitHub (Jun 12, 2022): Fixed in https://github.com/binwiederhier/ntfy/commit/41096ef1b06eff9624612fae28a1fc740b5af6dd If you can, why don't you try to create a pull request for things like this (docs updates, simple changes) next time. That'd help me a lot :-) If not, don't worry about it. Thank you for reporting this (and the other tickets!!)

BreizhHardware commented

2026-05-07 00:22:19 +02:00

Author

Owner

@milksteakjellybeans commented on GitHub (Jun 12, 2022):

sure, I will try to figure it out, I am new to GitHub

@milksteakjellybeans commented on GitHub (Jun 12, 2022): sure, I will try to figure it out, I am new to GitHub

BreizhHardware commented

2026-05-07 00:22:19 +02:00

Author

Owner

@binwiederhier commented on GitHub (Jun 12, 2022):

This will probably help: https://www.youtube.com/watch?v=rgbCcBNZcdQ

@binwiederhier commented on GitHub (Jun 12, 2022): This will probably help: https://www.youtube.com/watch?v=rgbCcBNZcdQ

BreizhHardware referenced this issue

2026-05-07 01:01:14 +02:00

[PR #252] [MERGED] Fix double listing of amd64 in the docs (Fixes #251) #1253