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

fix(mta): use swaks as an alternate to s-nail #2194

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

GMishx
Copy link
Member

@GMishx GMishx commented Mar 29, 2022

Description

Use swaks as an alternate to s-nail for MTA.

s-nail from past had many issues, there were some compatibility between versions causing troubles. The method used to design the authentication URL is not very good and causes issue with different setup. swaks provide different options to provide authentication information and server location which helps in separating the logic.

swaks looks like a good candidate to replace s-nail.

Changes

  1. Update the command generation string from s-nail format to swaks.
  2. Change the options of SMTPSslVerify from 'Ignore{I}|Strict{S}|Warn{W}' to simple 'Yes{1}|No{2}'.

How to test

  1. Setup email configuration for your server.
  2. Run a job and see if you receive email.

Closes #2240

GMishx and others added 2 commits June 1, 2022 15:58
Child is forked successfully, but can not write to it or read. While
child finishes, SIGCHLD is read by scheduler internal instead of GLib.

Signed-off-by: Gaurav Mishra <[email protected]>
@github-actions github-actions bot added the has merge conflicts PR to be rebased label Aug 1, 2022
@github-actions
Copy link

github-actions bot commented Aug 1, 2022

This pull request has conflicts, please rebase with master to resolve those before we can evaluate the pull request.

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