[Tickets #12736] Re: Automatic S/MIME encryption

noreply at bugs.horde.org noreply at bugs.horde.org
Mon Mar 13 21:05:45 UTC 2017


Ticket URL: https://bugs.horde.org/ticket/12736
  Ticket             | 12736
  Updated By         | jnaegele at grierforensics.com
  Summary            | Automatic S/MIME encryption
  Queue              | IMP
  Version            | Git master
  Type               | Enhancement
  State              | Accepted
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
+New Attachment     | 0001-Add-optional-automatic-S-MIME-encryption.patch

jnaegele at grierforensics.com (2017-03-13 21:05) wrote:

>> The if-clause in line 55 of your patch doesn't make sense because it
>> can never be true, see the check in 39.
>> This is a great start, thanks a lot! To be applied we would need at
>> least one of the two suggested solution for notifying about missing
>> encryption keys for certain recipients.
> Sure, the check in line 39 ensures that encryption is *not* already  
> enabled before attempting automatic encryption. Line 55 ensures that  
> if S/MIME signing is enabled it remains enabled in addition to  
> encryption.
> I'm happy to contribute the remainder of the patch but I'm not sure  
> how to best implement either option for notifying the user. I also  
> posted on the dev mailing list for tips. Thanks!

I've attached an updated patch that satisfies the discussed  
requirements. Existing functionality changes only when automatic  
S/MIME encryption is enabled in the S/MIME preference group.

If public keys for all recipients are found, the message will  
automatically be encrypted before being sent. Otherwise, the user will  
be notified of the recipients for which a public key could not be  
found. This notification will occur only one time, unless the user  
adds additional recipients for which public keys can't be found, in  
which case the user will be again notified. The notification is  
identical to the one used for detecting missing attachments.

Let me know what you think. This patch is actually pretty simple and  
applies cleanly.

jnaegele at grierforensics.com (2017-03-13 21:05) uploaded:  


More information about the bugs mailing list