Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[REQUEST] Add Instance Name to Discord notifications #1879

Closed
SawkeeReemo opened this issue Dec 4, 2024 · 9 comments · Fixed by #1884
Closed

[REQUEST] Add Instance Name to Discord notifications #1879

SawkeeReemo opened this issue Dec 4, 2024 · 9 comments · Fixed by #1884
Labels
🧹 chore Nothing fancy, just needs to get done

Comments

@SawkeeReemo
Copy link

SawkeeReemo commented Dec 4, 2024

Due to some limitations with other services like GlueTUN, I have multiple instances of Speed-Tester running. But there’s no way (that I’ve noticed) to discern which instance of Speed-Tester the alert is for. With email notifications, I can add the instance name to the email subject in the env vars, which is helpful. But I don’t see a way to do that for Discord, for example.

@alexjustesen alexjustesen added the 🧹 chore Nothing fancy, just needs to get done label Dec 4, 2024
@svenvg93
Copy link
Contributor

svenvg93 commented Dec 5, 2024

In the meantime until we add it to the notification a quick solution might be to make diffrent webhooks per instance in Discord and give theses the needed names.

image

@SawkeeReemo
Copy link
Author

Thanks for that! Thats a great tip for folks who pay for Discord. I actually tried that before posting here and learned that they only allow one webhook per free account.

Frankly speaking, I’d rather support you than them. I’m happy to wait for any potential addition for this. I just thought I’d make the request here so it was hopefully “on the books.” Really appreciate all the work you independent devs do. It’s inspired me to start learning how to code as well.

@svenvg93
Copy link
Contributor

svenvg93 commented Dec 5, 2024

Really? Im able to make multiple webhooks on a channel and I'm on a free account as well.
image

image

Still thought it won't be difficult to add the App name to the notification. Just need to find a good place to add it into the message 😄

@SawkeeReemo
Copy link
Author

Oh weird! It wouldn’t let me. I had to nuke the one I had in there before it would let me make a new one. The old one, I don’t remember what it was for haha… fingers crossed!

@SawkeeReemo
Copy link
Author

SawkeeReemo commented Dec 9, 2024

So… I’m not entirely sure my note was understood, because now all instance of Speedtest Tracker just have that exact text… so I still can’t tell which one the notification is for. I have 4 docker containers of Speedtest Tracker running on a NAS for four separate GlueTUN containers. In the email notifications, with the MAIL_FROM_NAME env car, I can type something like “SP1” so that when I get the email notification for threshold breech, I know which one it is.

Unfortunately, I haven’t seen a way to do that with the Discord notifications. So other than me remembering in which order I’ve scheduled them and if I even have all four running at the time… see what I mean?

EDIT: Weirdly Discord is now letting me make more webhooks when before it wouldn’t (even double check to make sure I wasn’t losing my mind 😅). Must have been a glitch and perfect timing. ¯_(ツ)_/¯

@svenvg93
Copy link
Contributor

svenvg93 commented Dec 10, 2024

You can change the "Speedtest Tracker" in the notification by setting the APP_NAME Env. With this it can be like.

A new speedtest on SP1 was completed using Ookla.

@SawkeeReemo
Copy link
Author

Oh! Thanks! I totally missed that env var. My mistake, and thank you. Love this program. Seriously nice work with it. …makes me want to figure out a way to automatically relaunch gluetun when my threshold limits hit.

@SawkeeReemo
Copy link
Author

Sorry, one more note: I just added “APP_NAME=MISC” to my compose, recreated the container, and ran a test. But it still just says Speedtest Tracker on Discord. Also not too worried about it since now that I’ve been able to make more webhooks, I just direct them to their own channels. But for the sake of bug reporting:
IMG_0539

and my compose:
https://hastebin.com/share/upolivaror.bash

@svenvg93
Copy link
Contributor

Hmm, do you see the change of the App_name in the webgui? Like in the top left corner?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 chore Nothing fancy, just needs to get done
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants