nextcloud / nextcloud/notify_push

systemd service file should include a "Restart=always"

Open
#194 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

documentation
Dominant language
Rust
Stars
344
Forks
64
Avg merge
2d 10h
Merged PRs (30d)
5

Description

When the client push is configured but the service is not running, this causes permanent browser issues as the browser always tries to establish a wss connection to the push service. You can see that in the browser console.

As a side effect this also delays the connection to the signaling server in Talk when entering a chat room ... up to messages that the signaling connection does not respond in time.

I found that, when the complete server VM is started, the notify_push service can start before the database is up. That leads to an error and notify_push often does not run.

The additional command

Restart=always

in the [Service] section should restart the service in that case until the database is up and the service is running.

Maybe that should be added in the documentation (sample service file).

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the notify_push systemd service file and inspect its [Service] section. Verify the service behavior when the database is unavailable during startup, then confirm that the unit retries until the service runs successfully.

Written by the indexing model from the issue text.

Assessment

Tech stack
linux
Domain
devops
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.