Skip to content

add dkim client capability#158

Open
gpatel-fr wants to merge 1 commit intoslimta:mainfrom
gpatel-fr:dkim
Open

add dkim client capability#158
gpatel-fr wants to merge 1 commit intoslimta:mainfrom
gpatel-fr:dkim

Conversation

@gpatel-fr
Copy link
Contributor

Hello

since I have found a bug in my implementation I am sending another attempt.

the bug is that when returning a delivery error, the sender is not defined, as no test was done in this case there was an inappropriate error message in the logs, hence this added code:

     dom = envelope.sender
     if not dom:
         return  # no warn if sending deliv. fail notif.

there are also a few cosmetic changes (pep8)

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant