You are here: Foswiki>Tasks Web>Item10522 (17 Dec 2011, GeorgeClark)Edit Attach

Item10522: Implement SMIME support

pencil
Priority: Enhancement
Current State: Closed
Released In: 1.1.4
Target Release: patch
Applies To: Engine
Component: FoswikiNet
Branches:
Reported By: GeorgeClark
Waiting For:
Last Change By: GeorgeClark
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:
smime.png

-- 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:
UI.png

-- 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
 
Topic revision: r26 - 17 Dec 2011, GeorgeClark
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy