Fix UptimeKuma url breaking (#526)

[dukeofsussex]

Description

Fix the UptimeKuma service infinitely appending the slug argument to the url.

Fixes #526

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• I've read & comply with the contributing guidelines
• I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.
• I have made corresponding changes to the documentation (README.md).
• I've checked my modifications for any breaking changes, especially in the config.yml file

[netlify]

✅ Deploy Preview for homer-demo-content ready!

Name	Link
🔨 Latest commit	0e2abd7
🔍 Latest deploy log	https://app.netlify.com/sites/homer-demo-content/deploys/63ecfafbff95db0008a97fca
😎 Deploy Preview	https://deploy-preview-603--homer-demo-content.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[jplanckeel-ep]

Up this subject :) We need fix 👍

[bastienwirtz]

Sorry for the very very late review, thanks for spotting this and fixing it 🙏

[bastienwirtz]

To avoid mutating props, I would suggest to remove all this, remove the slug property (need documentation update too), to use the common endpoint property. If you remove those line and configure the service this way:

- name: "Uptime Kuma"
  url: "http://192.168.0.151:3001"
  endpoint: "http://192.168.0.151:3001/status/default" # Replace default by your dashboard name if any
  type: "UptimeKuma"

instead of

- name: "Uptime Kuma"
  url: "http://192.168.0.151:3001"
  slug: "myCustomDashboard" # Defaults to "default" if not provided.
  type: "UptimeKuma"

It should works fine.

[dukeofsussex]

I agree mutating props isn't great, however your suggestion doesn't seem feasible, as endpoint is required to be a base path for the status queries to work

homer/src/components/services/UptimeKuma.vue

Lines 120 to 131 in b168430

	fetchStatus: function () {
	const now = Date.now()
	this.fetch(`/api/status-page/${this.dashboard}?cachebust=${now}`)
	.catch((e) => console.error(e))
	.then((resp) => (this.incident = resp));
	this.fetch(
	`/api/status-page/heartbeat/${this.dashboard}?cachebust=${now}`
	)
	.catch((e) => console.error(e))
	.then((resp) => (this.heartbeat = resp));
	},

homer/src/mixins/service.js

Lines 43 to 47 in b168430

	let url = this.endpoint;
	if (path) {
	url = `${this.endpoint}/${path}`;
	}

[bastienwirtz]

@jplanckeel-ep You can try to set the endpoint property in the configuration, it should work even without any code change. Something like

- name: "Uptime Kuma"
  url: "http://192.168.0.151:3001"
  endpoint: "http://192.168.0.151:3001/status/default" # Replace default by your dashboard name if any
  type: "UptimeKuma"

Can you confirm if it works§ Thanks 🙏