Skip to content

Fixes admin-1328 - Document web push notification requirements - #935

Merged
ralf401 merged 2 commits into
pre-releasefrom
rsc/fixes/admin-1328
Oct 14, 2026
Merged

ralf401 merged 2 commits into
pre-releasefrom
rsc/fixes/admin-1328

Conversation

@ralf401

@ralf401 ralf401 commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

The external connections list on the proxy and connections page tells operators which addresses Zammad has to reach. Web push notifications add another destination: the push service of the agent's browser.

Add a Web Push Notifications section with the addresses of the push services, the note that the proxy and no-proxy configuration applies to them, that Zammad needs no configuration (the VAPID key pair is generated once per installation and a change invalidates every device subscription) and that the vendor push service delivers the messages.

Relates to zammad/zammad-admin-documentation#1328

@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Artifacts

Produced during runtime

📦 html-da4009e

@ralf401

ralf401 commented Oct 12, 2026

Copy link
Copy Markdown
Contributor Author

The external connections list on the proxy and connections page tells
operators which addresses Zammad has to reach. Web push notifications add
another destination: the push service of the agent's browser.

Add a Web Push Notifications section with the addresses of the push
services, the note that the proxy and no-proxy configuration applies to
them, that Zammad needs no configuration (the VAPID key pair is generated
once per installation and a change invalidates every device subscription)
and that the vendor push service delivers the messages.

Verified against origin/develop of zammad/zammad:
- app/models/push_subscription.rb:11 (PUSH_SERVICE_DOMAINS; subdomains
  accepted, subscriptions of other push services rejected)
- app/services/service/user/push_subscription/deliver.rb:7 (TTL of one
  day), :83 (proxy and no-proxy list of the instance via
  UserAgent::HttpClient)
- db/migrate/20261008110019_web_push_vapid_keys.rb:8 and
  db/seeds/settings.rb:6582 (generated key pair, protected setting)
- app/frontend/apps/mobile/sw/sw.ts:69 (iOS ignores the push tag and
  stacks the notifications of one ticket)

Relates to zammad/zammad-admin-documentation#1328
Web push does not require the mobile view to be installed as an app in
general. The frontend reports the install reason as 'not-installed' only
for Apple mobile devices without a standalone app, on Android a supported
browser can receive the notifications as well.

Also fix a British spelling of 'optimization' in the same section.
@ralf401
ralf401 force-pushed the rsc/fixes/admin-1328 branch from 9637889 to 53feedf Compare October 13, 2026 09:34
@ralf401
ralf401 requested a review from suelje October 14, 2026 08:43
@ralf401
ralf401 merged commit ad3ea09 into pre-release Oct 14, 2026
1 check passed
@ralf401
ralf401 deleted the rsc/fixes/admin-1328 branch October 14, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants