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

Supporting notifications per backup configuration #55

Open
dataoscar opened this issue Feb 8, 2021 · 0 comments · May be fixed by #56
Open

Supporting notifications per backup configuration #55

dataoscar opened this issue Feb 8, 2021 · 0 comments · May be fixed by #56

Comments

@dataoscar
Copy link

I am in the process of adding HTTP notification support, but I am having some trouble since notifications are treated as global configurations. The net effect is that it makes it hard to have per-backup URLs and to integrate with services like healthchecks.io .

One of the simplest solutions would be to have per-backup notifications. A similar type of backup automation, Borgmatic, uses hooks per configuration. Have there been thoughts of going this route before?

dataoscar added a commit to dataoscar/duplicacy-util that referenced this issue Feb 10, 2021
Add an additional notifier that supports http requests using
GET, POST, and PUT methods.

This additionally adds support for declaring additional
notifications at the backup configuration. This is helpful
when HTTP notifications would be different per backup job and
would otherwise use different URLs.

Fixes jeffaco#51, closes jeffaco#55
@dataoscar dataoscar linked a pull request Feb 10, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant