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

[FEAT] Support for ntfy notifications #433

Closed
Fallenbagel opened this issue Jan 23, 2023 · 5 comments · Fixed by #471 or #474
Closed

[FEAT] Support for ntfy notifications #433

Fallenbagel opened this issue Jan 23, 2023 · 5 comments · Fixed by #471 or #474
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@Fallenbagel
Copy link

Is your feature request related to a problem? Please describe.
I would like to be able to get notifications on ntfy about smart failures.

Describe the solution you'd like
Be able to receive ntfy notifications about smart failures on my devices.

Additional context
Add any other context or screenshots about the feature request here.
Ntfy (pronounce: notify) is an open sourced simple HTTP-based pub-sub notification service that allows notifications on phone, desktop.

@Fallenbagel Fallenbagel changed the title [FEAT] Support for ntfy notificatkons [FEAT] Support for ntfy notifications Jan 23, 2023
@AnalogJ
Copy link
Owner

AnalogJ commented Jan 23, 2023

I see references to ntfy in the shoutrrr codebase (which is what Scrutiny uses for handling notifications to external services) -- it may already be supported, however their documentation site implies its only available in the dev branch https://containrrr.dev/shoutrrr/dev/services/ntfy/

@Robin-Sch
Copy link

Robin-Sch commented Apr 1, 2023

You can use a webhook for ntfy like this:

version: 1
notify:
    urls:
        - "https://ntfy.sh/xxxx"

For authorization (with user:pass or a token), see https://docs.ntfy.sh/publish/#query-param

@Fallenbagel
Copy link
Author

You can use a webhook for ntfy like this:

version: 1
notify:
    urls:
        - "https://ntfy.sh/xxxx"

For authorization (with user:pass or a token), see https://docs.ntfy.sh/publish/#query-param

Oh yeah that works! Although the payload is sent as is so no formatting right? I think for the time being I'll just create a webhook server and then divert the payload to ntfy for the time being 🤔

@Robin-Sch
Copy link

Yeah it's sent as JSON, maybe there should be/is an option to just send it as text?

@AnalogJ
Copy link
Owner

AnalogJ commented Apr 5, 2023

looks like shoutrrr v7.1 has been released. This should be a fairly easy upgrade (and will natively support ntfy).

@AnalogJ AnalogJ added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Apr 5, 2023
adamantike added a commit to adamantike/scrutiny that referenced this issue Apr 6, 2023
Updates [`shoutrrr`](containrrr.dev/shoutrrr/) to `v0.7.1` to enable
support for [ntfy](https://ntfy.sh/) notifications.

Fixes AnalogJ#433.
@AnalogJ AnalogJ moved this to Done in Scrutiny On-Deck Apr 7, 2023
@AnalogJ AnalogJ mentioned this issue Apr 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants