Bug: Self-hosted healthchecks.io does not work with additional paths #400

Closed
opened 2025-11-20 04:23:36 -05:00 by saavagebueno · 3 comments
Owner

Originally created by @Laptop765 on GitHub (May 14, 2024).

TLDR: Describe your issue in a one liner here

  1. Is this urgent: No

  2. DNS provider(s) you use: N/A

  3. Program version:

    Running version latest built on 2024-05-06T11:16:06.246Z (commit 542e895)

  4. What are you using to run the container: docker-compose

  5. Extra information (optional)

Self-hosted instances of healthchecks.io may have a ping url of the form ${BASE_URL}/ping/${UUID} which doesn't work with the implementation in 937a249ffa. I'm working on a PR to add this functionality.

Originally created by @Laptop765 on GitHub (May 14, 2024). <!-- YOU CAN CHAT THERE EVENTUALLY: https://github.com/qdm12/ddns-updater/discussions --> **TLDR**: *Describe your issue in a one liner here* 1. Is this urgent: No 2. DNS provider(s) you use: N/A 3. Program version: <!-- See the line at the top of your logs --> `Running version latest built on 2024-05-06T11:16:06.246Z (commit 542e895)` 4. What are you using to run the container: docker-compose 5. Extra information (optional) Self-hosted instances of healthchecks.io may have a ping url of the form `${BASE_URL}/ping/${UUID}` which doesn't work with the implementation in 937a249ffac9c7b498bcf7b90936f1228a049771. I'm working on a PR to add this functionality.
Author
Owner

@RubenHCCH commented on GitHub (May 14, 2024):

I had the same problem, I simply added the path /ping in HEALTH_HEALTHCHECKSIO_BASE_URL. Maybe documenting this is enough, and with this it is not necessary to add a new environment variable.

@RubenHCCH commented on GitHub (May 14, 2024): I had the same problem, I simply added the path `/ping` in `HEALTH_HEALTHCHECKSIO_BASE_URL`. Maybe documenting this is enough, and with this it is not necessary to add a new environment variable.
Author
Owner

@Laptop765 commented on GitHub (May 14, 2024):

Oh that's much simpler, I didn't think to try that 😅

@Laptop765 commented on GitHub (May 14, 2024): Oh that's much simpler, I didn't think to try that 😅
Author
Owner

@Laptop765 commented on GitHub (May 15, 2024):

Appending /ping to HEALTH_HEALTHCHECKSIO_BASE_URL worked for me, too. Going to close this and my PR 🙂

@Laptop765 commented on GitHub (May 15, 2024): Appending `/ping` to `HEALTH_HEALTHCHECKSIO_BASE_URL` worked for me, too. Going to close this and my PR 🙂
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ddns-updater#400