mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[GH-ISSUE #765] NTFY_WEB_ROOT: setting an alternative web root means any URL used gets a subscription added by that name, is this by design? #552
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#552
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?
Originally created by @mannp on GitHub (Jun 2, 2023).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/765
Hi there
I have configured my self-hosted instance with the following config;
The strange thing is I enter the url: https://ntfy.${SECRET_PROD_DOMAIN_XYZ}/ghdfdfghdfjk
I get the ui displayed with a subscription added named: ghdfdfghdfjk
Is this by design, as I would like to have / open and /app the url for the ui.
Thanks in advance
@binwiederhier commented on GitHub (Jun 3, 2023):
This is by design. It is a good assumption that if you go the the URL of a topic that you want to see the topic, no?
@mannp commented on GitHub (Jun 3, 2023):
If I had changed the root url to access the ui to /app I would not expect to access a subscription at / no.
The instance is deny all with its ui root url changed to /app, I expected it to do nothing or 404 at /
@binwiederhier commented on GitHub (Jun 12, 2023):
When it's deny-all I agree that it's a bit odd. It's also odd from a UX perspective that there is no "login screen" by default if it's set to deny-all.
@nwhistler commented on GitHub (Sep 11, 2023):
Yes, I would prefer that either the setting
deny-allorenable-loginwould force all browser traffic to/loginunless otherwise logged in. Is this something we should open a feature request for?