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

Shoutrrr Notification to Pushover doesn't respect Priority #1258

Closed
Snuffy2 opened this issue Mar 26, 2022 · 3 comments · Fixed by #1271
Closed

Shoutrrr Notification to Pushover doesn't respect Priority #1258

Snuffy2 opened this issue Mar 26, 2022 · 3 comments · Fixed by #1271
Assignees

Comments

@Snuffy2
Copy link

Snuffy2 commented Mar 26, 2022

Running Watchtower setup to send notifications to Pushover via Shoutrrr. It is working but the Priority doesn't seem to be respected. I'm trying to send it as a priority of -2 as I like to get it as an FYI but don't want an actual notification.

When I use Shoutrrr to call Pushover directly, then the Priority works as expected. Cross-posted as a Shoutrrr issue (containrrr/shoutrrr#230) as well. Happy to close one if desired.

From my docker-compose for Watchtower:

      - WATCHTOWER_NOTIFICATIONS=shoutrrr
      - "WATCHTOWER_NOTIFICATION_URL=pushover://shoutrrr:<mytoken>@<myuser>/?priority=-2"
      - "WATCHTOWER_NOTIFICATION_TEMPLATE=({{.Level}}): {{.Message}}{{println}}{{end}}"
@github-actions
Copy link

Hi there! 👋🏼 As you're new to this repo, we'd like to suggest that you read our code of conduct as well as our contribution guidelines. Thanks a bunch for opening your first issue! 🙏

@piksel
Copy link
Member

piksel commented Mar 28, 2022

Watchtower currently uses v0.5.2 of shoutrrr which was released before this fix was incorporated. We should release a new version with the fix included.

@piksel piksel self-assigned this Mar 28, 2022
@OliverHi
Copy link

OliverHi commented Aug 4, 2022

Any plans when this will be updated? I see that PR is already merged and would love to have properly working notification priorities. Please let me know if there is anything I can do to help make that happen

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

Successfully merging a pull request may close this issue.

3 participants