[PR #1434] [MERGED] Require login #1607

New issue

Closed

opened 2026-05-07 01:02:53 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/binwiederhier/ntfy/pull/1434
Author: @binwiederhier
Created: 8/24/2025
Status: ✅ Merged
Merged: 8/25/2025
Merged by: @binwiederhier

Base: main ← Head: require-login

📝 Commits (7)

03aeb70 feat: Add optional web app flag which requires a login for every action
569d89e Require login
ec1f97b Merge remote-tracking branch 'theatischbein/feat_optional_require_login' into require-login
3de04b2 Redirect to login page if require-login is enabled
4f6f45a Checks
e08f367 Fix lint
50f3563 Docs

📊 Changes

11 files changed (+59 additions, -30 deletions)

View changed files

📝 cmd/serve.go (+7 -2)
📝 docs/config.md (+1 -0)
📝 docs/releases.md (+4 -0)
📝 server/config.go (+2 -0)
📝 server/server.go (+3 -2)
📝 server/server.yml (+2 -0)
📝 server/types.go (+1 -0)
📝 web/public/config.js (+1 -0)
📝 web/src/app/utils.js (+24 -24)
📝 web/src/components/App.jsx (+7 -1)
📝 web/src/components/Notifications.jsx (+7 -1)

📄 Description

Closes #238, includes #1329

If require-login: true is set, the web app will redirect to the login page if no user was logged in

_{🔄 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/binwiederhier/ntfy/pull/1434 **Author:** [@binwiederhier](https://github.com/binwiederhier) **Created:** 8/24/2025 **Status:** ✅ Merged **Merged:** 8/25/2025 **Merged by:** [@binwiederhier](https://github.com/binwiederhier) **Base:** `main` ← **Head:** `require-login` --- ### 📝 Commits (7) - [`03aeb70`](https://github.com/binwiederhier/ntfy/commit/03aeb707f2a3276239765292d37311786b44e4b4) feat: Add optional web app flag which requires a login for every action - [`569d89e`](https://github.com/binwiederhier/ntfy/commit/569d89e8f897790a4aa2e5b82f8c9136c7cd1310) Require login - [`ec1f97b`](https://github.com/binwiederhier/ntfy/commit/ec1f97b72608f98631fc5c02c81f74518f216b01) Merge remote-tracking branch 'theatischbein/feat_optional_require_login' into require-login - [`3de04b2`](https://github.com/binwiederhier/ntfy/commit/3de04b27aba3dba8a36f21f1d796adab05050700) Redirect to login page if require-login is enabled - [`4f6f45a`](https://github.com/binwiederhier/ntfy/commit/4f6f45a9c06e224a30c44a66df9ce039a7b7fb9e) Checks - [`e08f367`](https://github.com/binwiederhier/ntfy/commit/e08f3670d1d5c5bf3d18053e7aad7792bb0e8ccf) Fix lint - [`50f3563`](https://github.com/binwiederhier/ntfy/commit/50f3563477b7850b438d52eb61bfeec530b77a38) Docs ### 📊 Changes **11 files changed** (+59 additions, -30 deletions) <details> <summary>View changed files</summary> 📝 `cmd/serve.go` (+7 -2) 📝 `docs/config.md` (+1 -0) 📝 `docs/releases.md` (+4 -0) 📝 `server/config.go` (+2 -0) 📝 `server/server.go` (+3 -2) 📝 `server/server.yml` (+2 -0) 📝 `server/types.go` (+1 -0) 📝 `web/public/config.js` (+1 -0) 📝 `web/src/app/utils.js` (+24 -24) 📝 `web/src/components/App.jsx` (+7 -1) 📝 `web/src/components/Notifications.jsx` (+7 -1) </details> ### 📄 Description Closes #238, includes #1329 If `require-login: true` is set, the web app will redirect to the login page if no user was logged in --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>