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

E-mail notification after backups #4

Open
niondir opened this issue Nov 29, 2016 · 14 comments
Open

E-mail notification after backups #4

niondir opened this issue Nov 29, 2016 · 14 comments
Assignees

Comments

@niondir
Copy link
Member

niondir commented Nov 29, 2016

No description provided.

@niondir niondir self-assigned this Nov 29, 2016
@mrclschstr
Copy link
Contributor

mrclschstr commented Jul 20, 2019

I forked your project to fit it to my personal needs. I implemendet a simple mail notification via mailx: see mrclschstr/restic-backup-docker@3068433 (not fully tested yet).

Maybe you can gain a little inspiration from that 😃

EDIT: Well, there is still a bug in my code. It seems like you cannot pass parameters to mailx using an environment variable. Executing the command directly with the given parameters seems to work. I'm still investigating...

EDIT 2: I think I found a fix (mrclschstr/restic-backup-docker@2e8faf3#diff-185accf3472a7ada6207586b6f31a4c0R46). Not sure if using a seperate shell execution has some downsides. Any help appreciated.

@niondir
Copy link
Member Author

niondir commented Jul 24, 2019

Cool I will have a look into it. If you are satisfied with your solution I would also happily accept a pull request.
As long if it's an optional feature that is off by default is see no risks.

@mrclschstr
Copy link
Contributor

Creating a PR is not that easy, because I already forked your project. I'll try to find a way around that...

@wuast94
Copy link

wuast94 commented Sep 17, 2019

How can I setup the email notification? I see this is merged so it should work right?

@mrclschstr
Copy link
Contributor

The readme contains an example configuration

@wuast94
Copy link

wuast94 commented Sep 18, 2019

I can't find it :/

@mrclschstr
Copy link
Contributor

You have to look into the alpine branch

@wuast94
Copy link

wuast94 commented Sep 18, 2019

OK but the alpine branch can't rclone?

@niondir
Copy link
Member Author

niondir commented Sep 19, 2019

Woops, I just noticed the two branches. We should probably merge them. I will have a look.

niondir added a commit that referenced this issue Sep 19, 2019
Merge Alpine to master for #4
@niondir
Copy link
Member Author

niondir commented Sep 19, 2019

I merged the alpine branch, since we are on alpine anyways. The integration of mailx is not as clean as it could be but should work.

I do not have time to test is yet. But if you do, I will provide a release on docker hub.

@wuast94
Copy link

wuast94 commented Sep 19, 2019

I will test it the next days. Can you make a docker build so that I can pull it?

@niondir
Copy link
Member Author

niondir commented May 18, 2020

@wuast94 it should be testable in "latest" tag. A new versioned tag will follow as well.

Edit: I added a new tag

@dreaming-sparrow
Copy link

dreaming-sparrow commented Sep 14, 2021

While testing the mail feature i always receive "send-mail: applet not found"
Do i need to install anything further into the container?

Solved!
Postfix was missing.
# apk add postfix
# postfix start
solved it for me!

@niondir
Copy link
Member Author

niondir commented Jan 19, 2022

If you can provide a PR that's very welcome.

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

No branches or pull requests

4 participants