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

Add Mail gateway #109

Merged
merged 5 commits into from
Oct 8, 2020
Merged

Add Mail gateway #109

merged 5 commits into from
Oct 8, 2020

Conversation

x-way
Copy link
Contributor

@x-way x-way commented Sep 30, 2020

Thanks for providing this nice ftpserver framework, I used it to build a basic FTP-to-Mail gateway which might be useful for others as well.

The story behind it is that we have a Brother printer-scanner-copier multifunctional device which is IP enabled.
It comes with a builtin Scan-to-FTP functionality but no Scan-to-Mail functionality.
With this basic FTP-to-Mail gateway in place there is now a one-button-press Scan-to-Mail functionality available 🙂

Additionally since everything is done inline, when the mail sending fails for some reason, this failure is reported back via the FTP response to the scanner device and immediately shown on the display to the user 🎉

@fclairamb
Copy link
Owner

Hi @x-way,
I love the idea !!!

The whole idea behind the ftpserver & ftpserverlib is to connect things from different decades. I was more thinking about connecting it to cloud services but emails are definitely a relevant use case.

And yes, I think reporting the SMTP sending status back to the FTP client at the end of transfer seems like a very smart choice.

Thank you for your contribution and for taking the time to go through my evil linting setup.

What comes to mind when I see this is that I should add the ability to write to two different "fs" / backend at once. So that we can both send it to email and archive it on S3 or SFTP.

@fclairamb fclairamb merged commit db91926 into fclairamb:master Oct 8, 2020
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 this pull request may close these issues.

2 participants