Skip to content

1.3.1 breaks availability monitoring #692

Description

@agross

I'm using Uptime Kuma to check for the healthiness and correctness of my notify_push setup. It involves reverse proxies and Redis, and there have been instances where a configuration change broke my setup and the monitor outlined below caught it instantly.

The monitor is a simple HTTP request (using Uptime Kuma, but this is unrelated):

$ curl https://nc.example.com/push/test/remote/1.2.3.4
1.2.3.4

The Uptime Kuma config also verifies 1.2.3.4 is returned.

With notify_push 1.3.1 in place, this request is now guarded by a token:

$ curl https://nc.example.com/push/test/remote/1.2.3.4
Missing request header "token"

I prefer to keep this simplistic but effective test. Uptime Kuma does not support executing arbitrary binaries like the provided test_client.

My suggestion is to add a configuration value that defines the value of a "test-only" token to be passed in the HTTP header.

Server configuration

Web server: Apache

Nextcloud version: 33

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions