nextcloud / nextcloud/notify_push
systemd service file should include a "Restart=always"
Nobody has claimed this yet.
- 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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