Item10522: Implement SMIME support
Priority: Enhancement
Current State: Closed
Released In: 1.1.4
Target Release: patch
Applies To: Engine
Component: FoswikiNet
Branches:
Implement
SMIMESupportInMail - Possibly port from (tm)Wiki.
--
GeorgeClark - 22 Mar 2011
Thanks to
TimotheLitt for the S/MIME code. Now committed to trunk.
--
GeorgeClark - 16 Jun 2011
Note: Below discussion moved from Item10873 so it will track with the task where the work is being done.
--
GeorgeClark - 16 Jun 2011
George -
Bad timing - I did the SMIME complete change (yours weren't in trunk earlier in the day). Since yours are only partial, I'm atttaching here in case it's useful. Sorry about the collision.
--
TimotheLitt - 15 Jun 2011
I've incorporated your changes and committed them to trunk. I did a bit more error checking to make the email code a bit more forgiving if the Certificate/Key are misconfigured. See
distro:705c624ae091. Thanks for providing the code. I've not tested it beyond error handling since I have not created any certificates yet.
--
GeorgeClark - 16 Jun 2011
Great. Thanks for the work. I'll svn update and give your version a whirl.
Also, it wasn't in the patch file, but if you go to
http://twiki.org/cgi-bin/view/TWiki/TWikiInstallationGuide, the doc update is the section "Enable Signed Email Notification".
You can get browser-recognized X509 certificates easily at startssl.com. Free (or cheap if you want them to look better.) No affiliation, but I recommend them.
--
TimotheLitt - 16 Jun 2011
That was easy. Works. One thing occurred to me as I reconfigured - it would be really nice if all the file/path stuff had a browser control. Beats all the typing. Maybe there's an AJAX approach - a bit of work to keep the variable substitution in the result.
S/MIME sent by Foswiki:
--
TimotheLitt - 16 Jun 2011
Small Nit: in the help text, we say
You must also use a mail program (not Net::SMTP) in the following settings . This isn't true any more - I think you want to say "Under the Email Server" tab.
I really DO like the new configure GUI..
--
TimotheLitt - 16 Jun 2011
Actually I only put the code for S/MIME under Sendmail for now. Until I have a test setup, I didn't want to add it into the Net::SMTP support. Ah... I found the text, and also noticed some T* references.
It would be best to put feedback about S/MIME support under
Item10522 where the work is committed. I'll try to copy some of this over.
--
GeorgeClark - 16 Jun 2011
Thanks for restoring the other topic to its original purpose.
What I meant here is that where the original text said "following settings", it was because they were further down (vertically) the screen of the old (linear) configure interface.
Now the mail program settings are on different
tab from the S/MIME settings.
So it would be clearer to point people to the tab, rather than "following" settings - nothing
follows S/MIME settings on the improved screen layout. That's a good thing, just suggesting that the on-screen doc match the on-screen layout.
Sorry about the T* references - I thought I caught those.
UI composite:
--
TimotheLitt - 16 Jun 2011
Sorry - I found and fixed all that. Since the bold text right under the S/MIME heading says that MailProgram is required, I deleted all of the obsolete text, and de-twikified it in
distro:4b7e5129fc54 - Drats... Missed the one under Certificate file - Fixed in
distro:41d87a56bc85. Note that the checker also complains if S/MIME is enabled with any other
MailMethod, so this is covered pretty well. InstallationGuide updated in
distro:9cfd4eff0b27
--
GeorgeClark - 17 Jun 2011