Version set to 1.0-alpha10-owntls

(closes #10)

include/Version.h

@@ -29,7 +29,7 @@
 // Product version as a single DWORD
 // Note: Used for version comparison within C/C++ code.
 //
-#define PRODUCT_VERSION          0x00ff0800
+#define PRODUCT_VERSION          0x00ff0a00
 
 //
 // Product version by components
@@ -39,26 +39,26 @@
 //
 #define PRODUCT_VERSION_MAJ      0
 #define PRODUCT_VERSION_MIN      255
-#define PRODUCT_VERSION_REV      8
+#define PRODUCT_VERSION_REV      10
 #define PRODUCT_VERSION_BUILD    0
 
 //
 // Human readable product version and build year for UI
 //
-#define PRODUCT_VERSION_STR      "1.0-alpha8"
+#define PRODUCT_VERSION_STR      "1.0-alpha10-owntls"
 #define PRODUCT_BUILD_YEAR_STR   "2016"
 
 //
 // Numerical version presentation for ProductVersion propery in
 // MSI packages (syntax: N.N[.N[.N]])
 //
-#define PRODUCT_VERSION_INST     "0.255.8"
+#define PRODUCT_VERSION_INST     "0.255.10"
 
 //
 // The product code for ProductCode property in MSI packages
 // Replace with new on every version change, regardless how minor it is.
 //
-#define PRODUCT_VERSION_GUID     "{82B292B6-F561-4DE1-8963-262A20B4E085}"
+#define PRODUCT_VERSION_GUID     "{C3675615-0D70-47C7-9BCB-B683A77C6ED6}"
 
 //
 // Since the product name is not finally confirmed at the time of

lib/EAPBase/include/Config.h

@@ -89,7 +89,6 @@ inline void operator>>(_Inout_ eap::cursor_in &cursor, _Out_ eap::config &val);
 #include <eaptypes.h> // Must include after <Windows.h>
 #include <tchar.h>
 
-#include <list>
 #include <string>
 #include <memory>
 
@@ -451,7 +450,7 @@ namespace eap
         winstd::tstring m_lbl_alt_credential;                   ///< Alternative label for credential prompt
         winstd::tstring m_lbl_alt_identity;                     ///< Alternative label for identity prompt
         winstd::tstring m_lbl_alt_password;                     ///< Alternative label for password prompt
-        std::list<std::unique_ptr<config_method> > m_methods;   ///< List of method configurations
+        std::vector<std::unique_ptr<config_method> > m_methods; ///< Array of method configurations
     };
 
 
@@ -551,7 +550,7 @@ namespace eap
         /// @}
 
     public:
-        std::list<eap::config_provider> m_providers;    ///< List of provider configurations
+        std::vector<eap::config_provider> m_providers;  ///< Array of provider configurations
     };
 }
 

lib/EAPBase/src/Config.cpp

@@ -296,7 +296,8 @@ eap::config_provider::config_provider(_In_ const config_provider &other) :
     m_lbl_alt_password(other.m_lbl_alt_password),
     config(other)
 {
-    for (list<unique_ptr<config_method> >::const_iterator method = other.m_methods.cbegin(), method_end = other.m_methods.cend(); method != method_end; ++method)
+    m_methods.reserve(other.m_methods.size());
+    for (vector<unique_ptr<config_method> >::const_iterator method = other.m_methods.cbegin(), method_end = other.m_methods.cend(); method != method_end; ++method)
         m_methods.push_back(std::move(unique_ptr<config_method>(*method ? (config_method*)method->get()->clone() : nullptr)));
 }
 
@@ -332,7 +333,8 @@ eap::config_provider& eap::config_provider::operator=(_In_ const config_provider
         m_lbl_alt_password   = other.m_lbl_alt_password;
 
         m_methods.clear();
-        for (list<unique_ptr<config_method> >::const_iterator method = other.m_methods.cbegin(), method_end = other.m_methods.cend(); method != method_end; ++method)
+        m_methods.reserve(other.m_methods.size());
+        for (vector<unique_ptr<config_method> >::const_iterator method = other.m_methods.cbegin(), method_end = other.m_methods.cend(); method != method_end; ++method)
             m_methods.push_back(std::move(unique_ptr<config_method>(*method ? (config_method*)method->get()->clone() : nullptr)));
     }
 
@@ -432,7 +434,7 @@ void eap::config_provider::save(_In_ IXMLDOMDocument *pDoc, _In_ IXMLDOMNode *pC
     if (FAILED(hr = eapxml::create_element(pDoc, pConfigRoot, bstr(L"eap-metadata:AuthenticationMethods"), bstr(L"AuthenticationMethods"), bstrNamespace, &pXmlElAuthenticationMethods)))
         throw com_runtime_error(hr, __FUNCTION__ " Error creating <AuthenticationMethods> element.");
 
-    for (list<unique_ptr<config_method> >::const_iterator method = m_methods.cbegin(), method_end = m_methods.cend(); method != method_end; ++method) {
+    for (vector<unique_ptr<config_method> >::const_iterator method = m_methods.cbegin(), method_end = m_methods.cend(); method != method_end; ++method) {
         // <AuthenticationMethod>
         com_obj<IXMLDOMElement> pXmlElAuthenticationMethod;
         if (FAILED(hr = eapxml::create_element(pDoc, bstr(L"AuthenticationMethod"), bstrNamespace, &pXmlElAuthenticationMethod)))
@@ -669,7 +671,7 @@ void eap::config_provider_list::save(_In_ IXMLDOMDocument *pDoc, _In_ IXMLDOMNod
     if (FAILED(hr = eapxml::select_node(pConfigRoot, bstr(L"eap-metadata:EAPIdentityProviderList"), &pXmlElIdentityProviderList)))
         throw com_runtime_error(hr, __FUNCTION__ " Error selecting <EAPIdentityProviderList> element.");
 
-    for (list<config_provider>::const_iterator provider = m_providers.cbegin(), provider_end = m_providers.cend(); provider != provider_end; ++provider) {
+    for (vector<config_provider>::const_iterator provider = m_providers.cbegin(), provider_end = m_providers.cend(); provider != provider_end; ++provider) {
         // <EAPIdentityProvider>
         com_obj<IXMLDOMElement> pXmlElIdentityProvider;
         if (FAILED(hr = eapxml::create_element(pDoc, bstr(L"EAPIdentityProvider"), bstrNamespace, &pXmlElIdentityProvider)))

lib/EAPBase_UI/include/EAP_UI.h

@@ -20,6 +20,7 @@
 
 #include <wx/hyperlink.h>
 #include <wx/icon.h>
+#include <wx/scrolwin.h>
 #include <wx/statbmp.h>
 #include <Windows.h>
 
@@ -34,18 +35,21 @@ class wxEAPBannerPanel;
 ///
 template <class _wxT> class wxEAPConfigDialog;
 
+///
+/// EAP general-use dialog
+///
+class wxEAPGeneralDialog;
+
 ///
 /// EAP top-most credential dialog
 ///
 class wxEAPCredentialsDialog;
 
-
 ///
 /// EAP general note
 ///
 class wxEAPNotePanel;
 
-
 ///
 /// EAP provider-locked congifuration note
 ///
@@ -56,6 +60,21 @@ class wxEAPProviderLockedPanel;
 ///
 class wxEAPCredentialWarningPanel;
 
+///
+/// EAP Configuration window
+///
+class wxEAPConfigWindow;
+
+///
+/// EAP provider identity config panel
+///
+class wxEAPProviderIdentityPanel;
+
+///
+/// EAP provider configuration dialog
+///
+class wxEAPConfigProvider;
+
 ///
 /// Base template for credential configuration panel
 ///
@@ -76,6 +95,11 @@ template <class _Tcred, class _Tbase> class wxPasswordCredentialsPanel;
 ///
 inline bool wxSetIconFromResource(wxStaticBitmap *bmp, wxIcon &icon, HINSTANCE hinst, PCWSTR pszName);
 
+///
+/// Returns GUI displayable provider name
+///
+inline wxString wxEAPGetProviderName(const std::wstring &id);
+
 #pragma once
 
 #include <wx/msw/winundef.h> // Fixes `CreateDialog` name collision
@@ -128,10 +152,10 @@ public:
         // Set extra style here, as wxFormBuilder overrides all default flags.
         this->SetExtraStyle(this->GetExtraStyle() | wxWS_EX_VALIDATE_RECURSIVELY);
 
-        for (std::list<eap::config_provider>::iterator provider = m_cfg.m_providers.begin(), provider_end = m_cfg.m_providers.end(); provider != provider_end; ++provider) {
+        for (std::vector<eap::config_provider>::iterator provider = m_cfg.m_providers.begin(), provider_end = m_cfg.m_providers.end(); provider != provider_end; ++provider) {
             bool is_single = provider->m_methods.size() == 1;
-            std::list<std::unique_ptr<eap::config_method> >::size_type count = 0;
+            std::vector<std::unique_ptr<eap::config_method> >::size_type count = 0;
-            std::list<std::unique_ptr<eap::config_method> >::iterator method = provider->m_methods.begin(), method_end = provider->m_methods.end();
+            std::vector<std::unique_ptr<eap::config_method> >::iterator method = provider->m_methods.begin(), method_end = provider->m_methods.end();
             for (; method != method_end; ++method, count++)
                 m_providers->AddPage(
                     new _wxT(
@@ -139,7 +163,9 @@ public:
                         *method->get(),
                         provider->m_id.c_str(),
                         m_providers),
-                    is_single ? provider->m_id : winstd::tstring_printf(_T("%s (%u)"), provider->m_id.c_str(), count));
+                    is_single ?
+                        wxEAPGetProviderName(provider->m_id) :
+                        winstd::tstring_printf(_T("%s (%u)"), wxEAPGetProviderName(provider->m_id), count));
         }
 
         this->Layout();
@@ -151,6 +177,7 @@ public:
 
 protected:
     /// \cond internal
+
     virtual void OnInitDialog(wxInitDialogEvent& event)
     {
         // Forward the event to child panels.
@@ -160,6 +187,22 @@ protected:
                 prov->GetEventHandler()->ProcessEvent(event);
         }
     }
+
+    virtual void OnUpdateUI(wxUpdateUIEvent& event)
+    {
+        UNREFERENCED_PARAMETER(event);
+
+        m_advanced->Enable(!m_cfg.m_providers.at(m_providers->GetSelection()).m_read_only);
+    }
+
+    virtual void OnAdvanced(wxCommandEvent& event)
+    {
+        UNREFERENCED_PARAMETER(event);
+
+        wxEAPConfigProvider dlg(m_cfg.m_providers.at(m_providers->GetSelection()), this);
+        dlg.ShowModal();
+    }
+
     /// \endcond
 
 
@@ -168,23 +211,38 @@ protected:
 };
 
 
-class wxEAPCredentialsDialog : public wxEAPCredentialsDialogBase
+class wxEAPGeneralDialog : public wxEAPGeneralDialogBase
+{
+public:
+    ///
+    /// Constructs a dialog
+    ///
+    wxEAPGeneralDialog(wxWindow* parent, const wxString& title = wxEmptyString);
+
+    ///
+    /// Adds panels to the dialog
+    ///
+    void AddContent(wxPanel **contents, size_t content_count);
+
+    ///
+    /// Adds single panel to the dialog
+    ///
+    void AddContent(wxPanel *content);
+
+protected:
+    /// \cond internal
+    virtual void OnInitDialog(wxInitDialogEvent& event);
+    /// \endcond
+};
+
+
+class wxEAPCredentialsDialog : public wxEAPGeneralDialog
 {
 public:
     ///
     /// Constructs a credential dialog
     ///
     wxEAPCredentialsDialog(const eap::config_provider &prov, wxWindow* parent);
-
-    ///
-    /// Adds panels to the dialog
-    ///
-    void AddContents(wxPanel **contents, size_t content_count);
-
-protected:
-    /// \cond internal
-    virtual void OnInitDialog(wxInitDialogEvent& event);
-    /// \endcond
 };
 
 
@@ -265,6 +323,103 @@ protected:
 };
 
 
+class wxEAPConfigWindow : public wxScrolledWindow
+{
+public:
+    ///
+    /// Constructs a configuration window
+    ///
+    /// \param[in]    prov    Provider configuration data
+    /// \param[inout] cfg     Configuration data
+    /// \param[in]    parent  Parent window
+    ///
+    wxEAPConfigWindow(const eap::config_provider &prov, eap::config_method &cfg, wxWindow* parent);
+
+    ///
+    /// Destructs the configuration window
+    ///
+    virtual ~wxEAPConfigWindow();
+
+protected:
+    /// \cond internal
+    virtual void OnInitDialog(wxInitDialogEvent& event);
+    virtual void OnUpdateUI(wxUpdateUIEvent& event);
+    /// \endcond
+
+protected:
+    const eap::config_provider &m_prov;     ///< EAP provider
+    eap::config_method &m_cfg;              ///< Method configuration
+};
+
+
+class wxEAPProviderIdentityPanel : public wxEAPProviderIdentityPanelBase
+{
+public:
+    ///
+    /// Constructs a provider identity pannel
+    ///
+    /// \param[inout] prov    Provider configuration data
+    /// \param[in]    parent  Parent window
+    ///
+    wxEAPProviderIdentityPanel(eap::config_provider &prov, wxWindow* parent);
+
+    friend class wxEAPConfigProvider; // Allows direct setting of keyboard focus
+
+protected:
+    /// \cond internal
+    virtual bool TransferDataToWindow();
+    virtual bool TransferDataFromWindow();
+    /// \endcond
+
+protected:
+    eap::config_provider &m_prov;   ///< EAP method configuration
+    winstd::library m_shell32;      ///< shell32.dll resource library reference
+    wxIcon m_icon;                  ///< Panel icon
+};
+
+
+class wxEAPProviderLockPanel : public wxEAPProviderLockPanelBase
+{
+public:
+    ///
+    /// Constructs a provider lock pannel
+    ///
+    /// \param[inout] prov    Provider configuration data
+    /// \param[in]    parent  Parent window
+    ///
+    wxEAPProviderLockPanel(eap::config_provider &prov, wxWindow* parent);
+
+protected:
+    /// \cond internal
+    virtual bool TransferDataToWindow();
+    virtual bool TransferDataFromWindow();
+    /// \endcond
+
+protected:
+    eap::config_provider &m_prov;   ///< EAP method configuration
+    winstd::library m_shell32;      ///< shell32.dll resource library reference
+    wxIcon m_icon;                  ///< Panel icon
+};
+
+
+class wxEAPConfigProvider : public wxEAPGeneralDialog
+{
+public:
+    ///
+    /// Constructs a provider config dialog
+    ///
+    /// \param[inout] prov    Provider configuration data
+    /// \param[in]    parent  Parent window
+    ///
+    wxEAPConfigProvider(eap::config_provider &prov, wxWindow* parent);
+
+protected:
+    eap::config_provider &m_prov;           ///< EAP method configuration
+    wxEAPProviderIdentityPanel *m_identity; ///< Provider identity panel
+    wxEAPProviderLockPanel *m_lock;         ///< Provider lock panel
+};
+
+
 template <class _Tcred, class _wxT>
 class wxEAPCredentialsConfigPanel : public wxEAPCredentialsConfigPanelBase
 {
@@ -289,6 +444,14 @@ public:
             wxSetIconFromResource(m_credentials_icon, m_icon, m_shell32, MAKEINTRESOURCE(/*16770*/269));
     }
 
+    ///
+    /// Sets keyboard focus to the first control that do not capture mouse wheel
+    ///
+    inline void SetFocusFromKbd()
+    {
+        m_own->SetFocusFromKbd();
+    }
+
 protected:
     /// \cond internal
 
@@ -402,7 +565,7 @@ protected:
         // Display credential prompt.
         wxEAPCredentialsDialog dlg(m_prov, this);
         _wxT *panel = new _wxT(m_prov, m_cfg, cred, m_target.c_str(), &dlg, true);
-        dlg.AddContents((wxPanel**)&panel, 1);
+        dlg.AddContent(panel);
         if (dlg.ShowModal() == wxID_OK && panel->GetRememberValue()) {
             // Write credentials to credential manager.
             try {
@@ -433,21 +596,21 @@ protected:
 
         _wxT *panel = new _wxT(m_prov, m_cfg, m_cred, _T(""), &dlg, true);
 
-        dlg.AddContents((wxPanel**)&panel, 1);
+        dlg.AddContent(panel);
         dlg.ShowModal();
     }
 
     /// \endcond
 
 protected:
-    const eap::config_provider &m_prov;             ///< EAP provider
+    const eap::config_provider &m_prov;     ///< EAP provider
     eap::config_method_with_cred &m_cfg;    ///< EAP method configuration
-    winstd::library m_shell32;                      ///< shell32.dll resource library reference
+    winstd::library m_shell32;              ///< shell32.dll resource library reference
-    wxIcon m_icon;                                  ///< Panel icon
+    wxIcon m_icon;                          ///< Panel icon
-    winstd::tstring m_target;                       ///< Credential Manager target
+    winstd::tstring m_target;               ///< Credential Manager target
 
 private:
-    _Tcred m_cred;                                  ///< Temporary credential data
+    _Tcred m_cred;                          ///< Temporary credential data
 };
 
 
@@ -581,12 +744,12 @@ protected:
         m_identity->SetSelection(0, -1);
         m_password->SetValue(m_cred.m_password.empty() ? wxEmptyString : s_dummy_password);
 
-        return wxEAPCredentialsPanelBase<_Tcred, wxEAPCredentialsPanelPassBase>::TransferDataToWindow();
+        return wxEAPCredentialsPanelBase<_Tcred, wxEAPCredentialsPassPanelBase>::TransferDataToWindow();
     }
 
     virtual bool TransferDataFromWindow()
     {
-        if (!wxEAPCredentialsPanelBase<_Tcred, wxEAPCredentialsPanelPassBase>::TransferDataFromWindow())
+        if (!wxEAPCredentialsPanelBase<_Tcred, wxEAPCredentialsPassPanelBase>::TransferDataFromWindow())
             return false;
 
         m_cred.m_identity = m_identity->GetValue();
@@ -609,7 +772,7 @@ protected:
             m_password      ->Enable(false);
         }
 
-        wxEAPCredentialsPanelBase<_Tcred, wxEAPCredentialsPanelPassBase>::OnUpdateUI(event);
+        wxEAPCredentialsPanelBase<_Tcred, wxEAPCredentialsPassPanelBase>::OnUpdateUI(event);
     }
 
     /// \endcond
@@ -638,3 +801,10 @@ inline bool wxSetIconFromResource(wxStaticBitmap *bmp, wxIcon &icon, HINSTANCE h
     } else
         return false;
 }
+
+
+inline wxString wxEAPGetProviderName(const std::wstring &id)
+{
+    return
+        !id.empty() ? id : _("<Your Organization>");
+}

lib/EAPBase_UI/res/wxEAP_UI.cpp

@@ -28,6 +28,20 @@ wxEAPConfigDialogBase::wxEAPConfigDialogBase( wxWindow* parent, wxWindowID id, c
 	
 	sb_content->Add( m_providers, 1, wxEXPAND|wxALL, 10 );
 	
+	wxBoxSizer* sb_bottom_horiz;
+	sb_bottom_horiz = new wxBoxSizer( wxHORIZONTAL );
+	
+	wxBoxSizer* sb_bottom_horiz_inner;
+	sb_bottom_horiz_inner = new wxBoxSizer( wxHORIZONTAL );
+	
+	m_advanced = new wxButton( this, wxID_ANY, _("Advanced..."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_advanced->SetToolTip( _("Opens dialog with provider settings") );
+	
+	sb_bottom_horiz_inner->Add( m_advanced, 0, wxALL, 5 );
+	
+	
+	sb_bottom_horiz->Add( sb_bottom_horiz_inner, 1, wxEXPAND, 5 );
+	
 	m_buttons = new wxStdDialogButtonSizer();
 	m_buttonsOK = new wxButton( this, wxID_OK );
 	m_buttons->AddButton( m_buttonsOK );
@@ -35,7 +49,10 @@ wxEAPConfigDialogBase::wxEAPConfigDialogBase( wxWindow* parent, wxWindowID id, c
 	m_buttons->AddButton( m_buttonsCancel );
 	m_buttons->Realize();
 	
-	sb_content->Add( m_buttons, 0, wxEXPAND|wxALL, 5 );
+	sb_bottom_horiz->Add( m_buttons, 0, wxEXPAND|wxALL, 5 );
+	
+	
+	sb_content->Add( sb_bottom_horiz, 0, wxEXPAND, 5 );
 	
 	
 	this->SetSizer( sb_content );
@@ -44,16 +61,20 @@ wxEAPConfigDialogBase::wxEAPConfigDialogBase( wxWindow* parent, wxWindowID id, c
 	
 	// Connect Events
 	this->Connect( wxEVT_INIT_DIALOG, wxInitDialogEventHandler( wxEAPConfigDialogBase::OnInitDialog ) );
+	this->Connect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPConfigDialogBase::OnUpdateUI ) );
+	m_advanced->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPConfigDialogBase::OnAdvanced ), NULL, this );
 }
 
 wxEAPConfigDialogBase::~wxEAPConfigDialogBase()
 {
 	// Disconnect Events
 	this->Disconnect( wxEVT_INIT_DIALOG, wxInitDialogEventHandler( wxEAPConfigDialogBase::OnInitDialog ) );
+	this->Disconnect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPConfigDialogBase::OnUpdateUI ) );
+	m_advanced->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPConfigDialogBase::OnAdvanced ), NULL, this );
 	
 }
 
-wxEAPCredentialsDialogBase::wxEAPCredentialsDialogBase( wxWindow* parent, wxWindowID id, const wxString& title, const wxPoint& pos, const wxSize& size, long style ) : wxDialog( parent, id, title, pos, size, style )
+wxEAPGeneralDialogBase::wxEAPGeneralDialogBase( wxWindow* parent, wxWindowID id, const wxString& title, const wxPoint& pos, const wxSize& size, long style ) : wxDialog( parent, id, title, pos, size, style )
 {
 	this->SetSizeHints( wxDefaultSize, wxDefaultSize );
 	
@@ -84,13 +105,13 @@ wxEAPCredentialsDialogBase::wxEAPCredentialsDialogBase( wxWindow* parent, wxWind
 	sb_content->Fit( this );
 	
 	// Connect Events
-	this->Connect( wxEVT_INIT_DIALOG, wxInitDialogEventHandler( wxEAPCredentialsDialogBase::OnInitDialog ) );
+	this->Connect( wxEVT_INIT_DIALOG, wxInitDialogEventHandler( wxEAPGeneralDialogBase::OnInitDialog ) );
 }
 
-wxEAPCredentialsDialogBase::~wxEAPCredentialsDialogBase()
+wxEAPGeneralDialogBase::~wxEAPGeneralDialogBase()
 {
 	// Disconnect Events
-	this->Disconnect( wxEVT_INIT_DIALOG, wxInitDialogEventHandler( wxEAPCredentialsDialogBase::OnInitDialog ) );
+	this->Disconnect( wxEVT_INIT_DIALOG, wxInitDialogEventHandler( wxEAPGeneralDialogBase::OnInitDialog ) );
 	
 }
 
@@ -99,20 +120,20 @@ wxEAPBannerPanelBase::wxEAPBannerPanelBase( wxWindow* parent, wxWindowID id, con
 	this->SetBackgroundColour( wxSystemSettings::GetColour( wxSYS_COLOUR_HIGHLIGHT ) );
 	this->SetMinSize( wxSize( -1,48 ) );
 	
-	wxBoxSizer* sc_content;
+	wxBoxSizer* sb_content;
-	sc_content = new wxBoxSizer( wxVERTICAL );
+	sb_content = new wxBoxSizer( wxVERTICAL );
 	
 	m_title = new wxStaticText( this, wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, wxALIGN_RIGHT );
 	m_title->Wrap( -1 );
 	m_title->SetFont( wxFont( 18, 70, 90, 90, false, wxEmptyString ) );
 	m_title->SetForegroundColour( wxSystemSettings::GetColour( wxSYS_COLOUR_HIGHLIGHTTEXT ) );
 	
-	sc_content->Add( m_title, 0, wxALL|wxEXPAND, 5 );
+	sb_content->Add( m_title, 0, wxALL|wxEXPAND, 5 );
 	
 	
-	this->SetSizer( sc_content );
+	this->SetSizer( sb_content );
 	this->Layout();
-	sc_content->Fit( this );
+	sb_content->Fit( this );
 }
 
 wxEAPBannerPanelBase::~wxEAPBannerPanelBase()
@@ -269,7 +290,7 @@ wxEAPCredentialsConfigPanelBase::~wxEAPCredentialsConfigPanelBase()
 	
 }
 
-wxEAPCredentialsPanelPassBase::wxEAPCredentialsPanelPassBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
+wxEAPCredentialsPassPanelBase::wxEAPCredentialsPassPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
 {
 	wxStaticBoxSizer* sb_credentials;
 	sb_credentials = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("Client Credentials") ), wxVERTICAL );
@@ -330,6 +351,168 @@ wxEAPCredentialsPanelPassBase::wxEAPCredentialsPanelPassBase( wxWindow* parent,
 	this->Layout();
 }
 
-wxEAPCredentialsPanelPassBase::~wxEAPCredentialsPanelPassBase()
+wxEAPCredentialsPassPanelBase::~wxEAPCredentialsPassPanelBase()
 {
 }
+
+wxEAPProviderIdentityPanelBase::wxEAPProviderIdentityPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
+{
+	wxStaticBoxSizer* sb_provider_id;
+	sb_provider_id = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("Your Organization") ), wxVERTICAL );
+	
+	wxBoxSizer* sb_provider_id_horiz;
+	sb_provider_id_horiz = new wxBoxSizer( wxHORIZONTAL );
+	
+	m_provider_id_icon = new wxStaticBitmap( sb_provider_id->GetStaticBox(), wxID_ANY, wxNullBitmap, wxDefaultPosition, wxDefaultSize, 0 );
+	sb_provider_id_horiz->Add( m_provider_id_icon, 0, wxALL, 5 );
+	
+	wxBoxSizer* sb_provider_id_vert;
+	sb_provider_id_vert = new wxBoxSizer( wxVERTICAL );
+	
+	m_provider_id_label = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _("Describe your organization to customize user prompts.  When organization is introduced, end-users find program messages easier to understand and act."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_id_label->Wrap( 446 );
+	sb_provider_id_vert->Add( m_provider_id_label, 0, wxALL|wxEXPAND, 5 );
+	
+	wxBoxSizer* sb_provider_name;
+	sb_provider_name = new wxBoxSizer( wxVERTICAL );
+	
+	m_provider_name_label = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _("Your organization &name:"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_name_label->Wrap( -1 );
+	sb_provider_name->Add( m_provider_name_label, 0, wxBOTTOM, 5 );
+	
+	m_provider_name = new wxTextCtrl( sb_provider_id->GetStaticBox(), wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_name->SetToolTip( _("Your organization name as it will appear on helpdesk contact notifications") );
+	
+	sb_provider_name->Add( m_provider_name, 0, wxEXPAND|wxBOTTOM, 5 );
+	
+	m_provider_name_note = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _("(Keep it short, please)"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_name_note->Wrap( -1 );
+	sb_provider_name->Add( m_provider_name_note, 0, wxALIGN_RIGHT, 5 );
+	
+	
+	sb_provider_id_vert->Add( sb_provider_name, 0, wxEXPAND|wxALL, 5 );
+	
+	wxBoxSizer* sb_provider_helpdesk;
+	sb_provider_helpdesk = new wxBoxSizer( wxVERTICAL );
+	
+	m_provider_helpdesk_label = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _("Helpdesk contact &information:"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_helpdesk_label->Wrap( -1 );
+	sb_provider_helpdesk->Add( m_provider_helpdesk_label, 0, wxBOTTOM, 5 );
+	
+	wxFlexGridSizer* sb_provider_helpdesk_inner;
+	sb_provider_helpdesk_inner = new wxFlexGridSizer( 0, 2, 0, 0 );
+	sb_provider_helpdesk_inner->AddGrowableCol( 1 );
+	sb_provider_helpdesk_inner->SetFlexibleDirection( wxBOTH );
+	sb_provider_helpdesk_inner->SetNonFlexibleGrowMode( wxFLEX_GROWMODE_SPECIFIED );
+	
+	m_provider_web_icon = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _("¶"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_web_icon->Wrap( -1 );
+	m_provider_web_icon->SetFont( wxFont( wxNORMAL_FONT->GetPointSize(), 70, 90, 90, false, wxT("Wingdings") ) );
+	
+	sb_provider_helpdesk_inner->Add( m_provider_web_icon, 0, wxALIGN_CENTER_VERTICAL|wxBOTTOM|wxRIGHT, 5 );
+	
+	m_provider_web = new wxTextCtrl( sb_provider_id->GetStaticBox(), wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_web->SetToolTip( _("Your helpdesk website") );
+	
+	sb_provider_helpdesk_inner->Add( m_provider_web, 1, wxEXPAND|wxALIGN_CENTER_VERTICAL|wxBOTTOM, 5 );
+	
+	m_provider_email_icon = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _("*"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_email_icon->Wrap( -1 );
+	m_provider_email_icon->SetFont( wxFont( wxNORMAL_FONT->GetPointSize(), 70, 90, 90, false, wxT("Wingdings") ) );
+	
+	sb_provider_helpdesk_inner->Add( m_provider_email_icon, 0, wxALIGN_CENTER_VERTICAL|wxBOTTOM|wxRIGHT, 5 );
+	
+	m_provider_email = new wxTextCtrl( sb_provider_id->GetStaticBox(), wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_email->SetToolTip( _("Your helpdesk e-mail address") );
+	
+	sb_provider_helpdesk_inner->Add( m_provider_email, 1, wxEXPAND|wxALIGN_CENTER_VERTICAL|wxBOTTOM, 5 );
+	
+	m_provider_phone_icon = new wxStaticText( sb_provider_id->GetStaticBox(), wxID_ANY, _(")"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_phone_icon->Wrap( -1 );
+	m_provider_phone_icon->SetFont( wxFont( wxNORMAL_FONT->GetPointSize(), 70, 90, 90, false, wxT("Wingdings") ) );
+	
+	sb_provider_helpdesk_inner->Add( m_provider_phone_icon, 0, wxALIGN_CENTER_VERTICAL|wxRIGHT, 5 );
+	
+	m_provider_phone = new wxTextCtrl( sb_provider_id->GetStaticBox(), wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_phone->SetToolTip( _("Your helpdesk phone number") );
+	
+	sb_provider_helpdesk_inner->Add( m_provider_phone, 1, wxEXPAND|wxALIGN_CENTER_VERTICAL, 5 );
+	
+	
+	sb_provider_helpdesk->Add( sb_provider_helpdesk_inner, 1, wxEXPAND, 5 );
+	
+	
+	sb_provider_id_vert->Add( sb_provider_helpdesk, 1, wxEXPAND, 5 );
+	
+	
+	sb_provider_id_horiz->Add( sb_provider_id_vert, 1, wxEXPAND, 5 );
+	
+	
+	sb_provider_id->Add( sb_provider_id_horiz, 1, wxEXPAND, 5 );
+	
+	
+	this->SetSizer( sb_provider_id );
+	this->Layout();
+	
+	// Connect Events
+	this->Connect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPProviderIdentityPanelBase::OnUpdateUI ) );
+}
+
+wxEAPProviderIdentityPanelBase::~wxEAPProviderIdentityPanelBase()
+{
+	// Disconnect Events
+	this->Disconnect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPProviderIdentityPanelBase::OnUpdateUI ) );
+	
+}
+
+wxEAPProviderLockPanelBase::wxEAPProviderLockPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
+{
+	wxStaticBoxSizer* sb_provider_lock;
+	sb_provider_lock = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("Configuration Lock") ), wxVERTICAL );
+	
+	wxBoxSizer* sb_provider_lock_horiz;
+	sb_provider_lock_horiz = new wxBoxSizer( wxHORIZONTAL );
+	
+	m_provider_lock_icon = new wxStaticBitmap( sb_provider_lock->GetStaticBox(), wxID_ANY, wxNullBitmap, wxDefaultPosition, wxDefaultSize, 0 );
+	sb_provider_lock_horiz->Add( m_provider_lock_icon, 0, wxALL, 5 );
+	
+	wxBoxSizer* sb_provider_lock_vert;
+	sb_provider_lock_vert = new wxBoxSizer( wxVERTICAL );
+	
+	m_provider_lock_label = new wxStaticText( sb_provider_lock->GetStaticBox(), wxID_ANY, _("Your configuration can be locked to prevent accidental modification by end-users. Users will only be allowed to enter credentials."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_lock_label->Wrap( 446 );
+	sb_provider_lock_vert->Add( m_provider_lock_label, 0, wxALL|wxEXPAND, 5 );
+	
+	wxBoxSizer* sb_provider_name;
+	sb_provider_name = new wxBoxSizer( wxVERTICAL );
+	
+	m_provider_lock = new wxCheckBox( sb_provider_lock->GetStaticBox(), wxID_ANY, _("&Lock this configuration and prevent any further modification via user interface."), wxDefaultPosition, wxDefaultSize, 0 );
+	sb_provider_name->Add( m_provider_lock, 0, wxEXPAND|wxBOTTOM, 5 );
+	
+	m_provider_lock_note = new wxStaticText( sb_provider_lock->GetStaticBox(), wxID_ANY, _("(Warning: Once locked, you can not revert using this dialog!)"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_provider_lock_note->Wrap( -1 );
+	sb_provider_name->Add( m_provider_lock_note, 0, wxALIGN_RIGHT, 5 );
+	
+	
+	sb_provider_lock_vert->Add( sb_provider_name, 0, wxEXPAND|wxALL, 5 );
+	
+	
+	sb_provider_lock_horiz->Add( sb_provider_lock_vert, 1, wxEXPAND, 5 );
+	
+	
+	sb_provider_lock->Add( sb_provider_lock_horiz, 1, wxEXPAND, 5 );
+	
+	
+	this->SetSizer( sb_provider_lock );
+	this->Layout();
+	
+	// Connect Events
+	this->Connect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPProviderLockPanelBase::OnUpdateUI ) );
+}
+
+wxEAPProviderLockPanelBase::~wxEAPProviderLockPanelBase()
+{
+	// Disconnect Events
+	this->Disconnect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPProviderLockPanelBase::OnUpdateUI ) );
+	
+}

lib/EAPBase_UI/res/wxEAP_UI.h

@@ -18,8 +18,8 @@ class wxEAPBannerPanel;
 #include <wx/settings.h>
 #include <wx/string.h>
 #include <wx/notebook.h>
-#include <wx/sizer.h>
 #include <wx/button.h>
+#include <wx/sizer.h>
 #include <wx/dialog.h>
 #include <wx/stattext.h>
 #include <wx/panel.h>
@@ -44,12 +44,15 @@ class wxEAPConfigDialogBase : public wxDialog
 	protected:
 		wxEAPBannerPanel *m_banner;
 		wxNotebook* m_providers;
+		wxButton* m_advanced;
 		wxStdDialogButtonSizer* m_buttons;
 		wxButton* m_buttonsOK;
 		wxButton* m_buttonsCancel;
 		
 		// Virtual event handlers, overide them in your derived class
 		virtual void OnInitDialog( wxInitDialogEvent& event ) { event.Skip(); }
+		virtual void OnUpdateUI( wxUpdateUIEvent& event ) { event.Skip(); }
+		virtual void OnAdvanced( wxCommandEvent& event ) { event.Skip(); }
 		
 	
 	public:
@@ -60,9 +63,9 @@ class wxEAPConfigDialogBase : public wxDialog
 };
 
 ///////////////////////////////////////////////////////////////////////////////
-/// Class wxEAPCredentialsDialogBase
+/// Class wxEAPGeneralDialogBase
 ///////////////////////////////////////////////////////////////////////////////
-class wxEAPCredentialsDialogBase : public wxDialog 
+class wxEAPGeneralDialogBase : public wxDialog 
 {
 	private:
 	
@@ -79,8 +82,8 @@ class wxEAPCredentialsDialogBase : public wxDialog
 	
 	public:
 		
-		wxEAPCredentialsDialogBase( wxWindow* parent, wxWindowID id = wxID_ANY, const wxString& title = _("EAP Credentials"), const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize, long style = wxDEFAULT_DIALOG_STYLE ); 
+		wxEAPGeneralDialogBase( wxWindow* parent, wxWindowID id = wxID_ANY, const wxString& title = wxEmptyString, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize, long style = wxDEFAULT_DIALOG_STYLE ); 
-		~wxEAPCredentialsDialogBase();
+		~wxEAPGeneralDialogBase();
 	
 };
 
@@ -153,9 +156,9 @@ class wxEAPCredentialsConfigPanelBase : public wxPanel
 };
 
 ///////////////////////////////////////////////////////////////////////////////
-/// Class wxEAPCredentialsPanelPassBase
+/// Class wxEAPCredentialsPassPanelBase
 ///////////////////////////////////////////////////////////////////////////////
-class wxEAPCredentialsPanelPassBase : public wxPanel 
+class wxEAPCredentialsPassPanelBase : public wxPanel 
 {
 	private:
 	
@@ -170,8 +173,64 @@ class wxEAPCredentialsPanelPassBase : public wxPanel
 	
 	public:
 		
-		wxEAPCredentialsPanelPassBase( wxWindow* parent, wxWindowID id = wxID_ANY, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxSize( 500,-1 ), long style = wxTAB_TRAVERSAL ); 
+		wxEAPCredentialsPassPanelBase( wxWindow* parent, wxWindowID id = wxID_ANY, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxSize( 500,-1 ), long style = wxTAB_TRAVERSAL ); 
-		~wxEAPCredentialsPanelPassBase();
+		~wxEAPCredentialsPassPanelBase();
+	
+};
+
+///////////////////////////////////////////////////////////////////////////////
+/// Class wxEAPProviderIdentityPanelBase
+///////////////////////////////////////////////////////////////////////////////
+class wxEAPProviderIdentityPanelBase : public wxPanel 
+{
+	private:
+	
+	protected:
+		wxStaticBitmap* m_provider_id_icon;
+		wxStaticText* m_provider_id_label;
+		wxStaticText* m_provider_name_label;
+		wxTextCtrl* m_provider_name;
+		wxStaticText* m_provider_name_note;
+		wxStaticText* m_provider_helpdesk_label;
+		wxStaticText* m_provider_web_icon;
+		wxTextCtrl* m_provider_web;
+		wxStaticText* m_provider_email_icon;
+		wxTextCtrl* m_provider_email;
+		wxStaticText* m_provider_phone_icon;
+		wxTextCtrl* m_provider_phone;
+		
+		// Virtual event handlers, overide them in your derived class
+		virtual void OnUpdateUI( wxUpdateUIEvent& event ) { event.Skip(); }
+		
+	
+	public:
+		
+		wxEAPProviderIdentityPanelBase( wxWindow* parent, wxWindowID id = wxID_ANY, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxSize( 500,-1 ), long style = wxTAB_TRAVERSAL ); 
+		~wxEAPProviderIdentityPanelBase();
+	
+};
+
+///////////////////////////////////////////////////////////////////////////////
+/// Class wxEAPProviderLockPanelBase
+///////////////////////////////////////////////////////////////////////////////
+class wxEAPProviderLockPanelBase : public wxPanel 
+{
+	private:
+	
+	protected:
+		wxStaticBitmap* m_provider_lock_icon;
+		wxStaticText* m_provider_lock_label;
+		wxCheckBox* m_provider_lock;
+		wxStaticText* m_provider_lock_note;
+		
+		// Virtual event handlers, overide them in your derived class
+		virtual void OnUpdateUI( wxUpdateUIEvent& event ) { event.Skip(); }
+		
+	
+	public:
+		
+		wxEAPProviderLockPanelBase( wxWindow* parent, wxWindowID id = wxID_ANY, const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxSize( 500,-1 ), long style = wxTAB_TRAVERSAL ); 
+		~wxEAPProviderLockPanelBase();
 	
 };
 

lib/EAPBase_UI/src/EAP_UI.cpp

@@ -38,22 +38,19 @@ bool wxEAPBannerPanel::AcceptsFocusFromKeyboard() const
 
 
 //////////////////////////////////////////////////////////////////////
-// wxEAPCredentialsDialog
+// wxEAPGeneralDialog
 //////////////////////////////////////////////////////////////////////
 
-wxEAPCredentialsDialog::wxEAPCredentialsDialog(const eap::config_provider &prov, wxWindow* parent) : wxEAPCredentialsDialogBase(parent)
+wxEAPGeneralDialog::wxEAPGeneralDialog(wxWindow* parent, const wxString& title) : wxEAPGeneralDialogBase(parent, wxID_ANY, title)
 {
     // Set extra style here, as wxFormBuilder overrides all default flags.
     this->SetExtraStyle(this->GetExtraStyle() | wxWS_EX_VALIDATE_RECURSIVELY);
 
-    // Set banner title.
-    m_banner->m_title->SetLabel(wxString::Format(_("%s Credentials"), prov.m_id.c_str()));
-
     m_buttonsOK->SetDefault();
 }
 
 
-void wxEAPCredentialsDialog::AddContents(wxPanel **contents, size_t content_count)
+void wxEAPGeneralDialog::AddContent(wxPanel **contents, size_t content_count)
 {
     if (content_count) {
         for (size_t i = 0; i < content_count; i++)
@@ -66,13 +63,30 @@ void wxEAPCredentialsDialog::AddContents(wxPanel **contents, size_t content_coun
 }
 
 
-void wxEAPCredentialsDialog::OnInitDialog(wxInitDialogEvent& event)
+void wxEAPGeneralDialog::AddContent(wxPanel *content)
+{
+    AddContent(&content, 1);
+}
+
+
+void wxEAPGeneralDialog::OnInitDialog(wxInitDialogEvent& event)
 {
     for (wxSizerItemList::compatibility_iterator panel = m_panels->GetChildren().GetFirst(); panel; panel = panel->GetNext())
         panel->GetData()->GetWindow()->GetEventHandler()->ProcessEvent(event);
 }
 
 
+//////////////////////////////////////////////////////////////////////
+// wxEAPCredentialsDialog
+//////////////////////////////////////////////////////////////////////
+
+wxEAPCredentialsDialog::wxEAPCredentialsDialog(const eap::config_provider &prov, wxWindow* parent) : wxEAPGeneralDialog(parent, _("EAP Credentials"))
+{
+    // Set banner title.
+    m_banner->m_title->SetLabel(wxString::Format(_("%s Credentials"), wxEAPGetProviderName(prov.m_id).c_str()));
+}
+
+
 //////////////////////////////////////////////////////////////////////
 // wxEAPNotePanel
 //////////////////////////////////////////////////////////////////////
@@ -189,3 +203,143 @@ wxEAPCredentialWarningPanel::wxEAPCredentialWarningPanel(const eap::config_provi
 
     this->Layout();
 }
+
+
+//////////////////////////////////////////////////////////////////////
+// wxEAPConfigWindow
+//////////////////////////////////////////////////////////////////////
+
+wxEAPConfigWindow::wxEAPConfigWindow(const eap::config_provider &prov, eap::config_method &cfg, wxWindow* parent) :
+    m_prov(prov),
+    m_cfg(cfg),
+    wxScrolledWindow(parent, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxVSCROLL)
+{
+    this->SetScrollRate(5, 5);
+
+    // Connect Events
+    this->Connect(wxEVT_INIT_DIALOG, wxInitDialogEventHandler(wxEAPConfigWindow::OnInitDialog));
+    this->Connect(wxEVT_UPDATE_UI, wxUpdateUIEventHandler(wxEAPConfigWindow::OnUpdateUI));
+}
+
+
+wxEAPConfigWindow::~wxEAPConfigWindow()
+{
+    // Disconnect Events
+    this->Disconnect(wxEVT_UPDATE_UI, wxUpdateUIEventHandler(wxEAPConfigWindow::OnUpdateUI));
+    this->Disconnect(wxEVT_INIT_DIALOG, wxInitDialogEventHandler(wxEAPConfigWindow::OnInitDialog));
+}
+
+
+void wxEAPConfigWindow::OnInitDialog(wxInitDialogEvent& event)
+{
+    UNREFERENCED_PARAMETER(event);
+
+    // Call TransferDataToWindow() manually, as wxScrolledWindow somehow skips that.
+    TransferDataToWindow();
+}
+
+
+void wxEAPConfigWindow::OnUpdateUI(wxUpdateUIEvent& event)
+{
+    UNREFERENCED_PARAMETER(event);
+
+    if (m_parent && m_parent->IsKindOf(wxCLASSINFO(wxNotebook))) {
+        // We're a notebook page. Set the ID of our provider as our page label.
+        wxNotebook *notebook = (wxNotebook*)m_parent;
+        int idx = notebook->FindPage(this);
+        if (idx != wxNOT_FOUND)
+            notebook->SetPageText(idx, wxEAPGetProviderName(m_prov.m_id));
+    } else
+        this->SetLabel(wxEAPGetProviderName(m_prov.m_id));
+}
+
+
+//////////////////////////////////////////////////////////////////////
+// wxEAPProviderIdentityPanel
+//////////////////////////////////////////////////////////////////////
+
+wxEAPProviderIdentityPanel::wxEAPProviderIdentityPanel(eap::config_provider &prov, wxWindow* parent) :
+    m_prov(prov),
+    wxEAPProviderIdentityPanelBase(parent)
+{
+    // Load and set icon.
+    if (m_shell32.load(_T("shell32.dll"), NULL, LOAD_LIBRARY_AS_DATAFILE | LOAD_LIBRARY_AS_IMAGE_RESOURCE))
+        wxSetIconFromResource(m_provider_id_icon, m_icon, m_shell32, MAKEINTRESOURCE(259));
+}
+
+
+bool wxEAPProviderIdentityPanel::TransferDataToWindow()
+{
+    m_provider_name ->SetValue(m_prov.m_id        );
+    m_provider_web  ->SetValue(m_prov.m_help_web  );
+    m_provider_email->SetValue(m_prov.m_help_email);
+    m_provider_phone->SetValue(m_prov.m_help_phone);
+
+    return wxEAPProviderIdentityPanelBase::TransferDataToWindow();
+}
+
+
+bool wxEAPProviderIdentityPanel::TransferDataFromWindow()
+{
+    wxCHECK(wxEAPProviderIdentityPanelBase::TransferDataFromWindow(), false);
+
+    m_prov.m_id         = m_provider_name ->GetValue();
+    m_prov.m_help_web   = m_provider_web  ->GetValue();
+    m_prov.m_help_email = m_provider_email->GetValue();
+    m_prov.m_help_phone = m_provider_phone->GetValue();
+
+    return true;
+}
+
+
+//////////////////////////////////////////////////////////////////////
+// wxEAPProviderLockPanel
+//////////////////////////////////////////////////////////////////////
+
+wxEAPProviderLockPanel::wxEAPProviderLockPanel(eap::config_provider &prov, wxWindow* parent) :
+    m_prov(prov),
+    wxEAPProviderLockPanelBase(parent)
+{
+    // Load and set icon.
+    if (m_shell32.load(_T("shell32.dll"), NULL, LOAD_LIBRARY_AS_DATAFILE | LOAD_LIBRARY_AS_IMAGE_RESOURCE))
+        wxSetIconFromResource(m_provider_lock_icon, m_icon, m_shell32, MAKEINTRESOURCE(1003));
+}
+
+
+bool wxEAPProviderLockPanel::TransferDataToWindow()
+{
+    m_provider_lock->SetValue(m_prov.m_read_only);
+
+    return wxEAPProviderLockPanelBase::TransferDataToWindow();
+}
+
+
+bool wxEAPProviderLockPanel::TransferDataFromWindow()
+{
+    wxCHECK(wxEAPProviderLockPanelBase::TransferDataFromWindow(), false);
+
+    m_prov.m_read_only = m_provider_lock->GetValue();
+
+    return true;
+}
+
+
+//////////////////////////////////////////////////////////////////////
+// wxEAPConfigProvider
+//////////////////////////////////////////////////////////////////////
+
+wxEAPConfigProvider::wxEAPConfigProvider(eap::config_provider &prov, wxWindow* parent) :
+    m_prov(prov),
+    wxEAPGeneralDialog(parent, _("Provider Settings"))
+{
+    // Set banner title.
+    m_banner->m_title->SetLabel(_("Provider Settings"));
+
+    m_identity = new wxEAPProviderIdentityPanel(prov, this);
+    AddContent(m_identity);
+
+    m_lock = new wxEAPProviderLockPanel(prov, this);
+    AddContent(m_lock);
+
+    m_identity->m_provider_name->SetFocusFromKbd();
+}

lib/PAP_UI/include/PAP_UI.h

@@ -25,7 +25,7 @@
 ///
 /// PAP credential configuration panel
 ///
-typedef wxEAPCredentialsConfigPanel<eap::credentials_pap, wxPasswordCredentialsPanel<eap::credentials_pap, wxEAPCredentialsPanelPassBase> > wxPAPCredentialsConfigPanel;
+typedef wxEAPCredentialsConfigPanel<eap::credentials_pap, wxPasswordCredentialsPanel<eap::credentials_pap, wxEAPCredentialsPassPanelBase> > wxPAPCredentialsConfigPanel;
 
 ///
 /// PAP configuration panel
@@ -35,7 +35,7 @@ class wxPAPConfigPanel;
 ///
 /// PAP credential entry panel
 ///
-typedef wxPasswordCredentialsPanel<eap::credentials_pap, wxEAPCredentialsPanelPassBase> wxPAPCredentialsPanel;
+typedef wxPasswordCredentialsPanel<eap::credentials_pap, wxEAPCredentialsPassPanelBase> wxPAPCredentialsPanel;
 
 #pragma once
 

lib/TLS/include/Config.h

@@ -1,176 +1,176 @@
-/*
+/*
-    Copyright 2015-2016 Amebis
+    Copyright 2015-2016 Amebis
-    Copyright 2016 GÉANT
+    Copyright 2016 GÉANT
-
+
-    This file is part of GÉANTLink.
+    This file is part of GÉANTLink.
-
+
-    GÉANTLink is free software: you can redistribute it and/or modify it
+    GÉANTLink is free software: you can redistribute it and/or modify it
-    under the terms of the GNU General Public License as published by
+    under the terms of the GNU General Public License as published by
-    the Free Software Foundation, either version 3 of the License, or
+    the Free Software Foundation, either version 3 of the License, or
-    (at your option) any later version.
+    (at your option) any later version.
-
+
-    GÉANTLink is distributed in the hope that it will be useful, but
+    GÉANTLink is distributed in the hope that it will be useful, but
-    WITHOUT ANY WARRANTY; without even the implied warranty of
+    WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
+    GNU General Public License for more details.
-
+
-    You should have received a copy of the GNU General Public License
+    You should have received a copy of the GNU General Public License
-    along with GÉANTLink. If not, see <http://www.gnu.org/licenses/>.
+    along with GÉANTLink. If not, see <http://www.gnu.org/licenses/>.
-*/
+*/
-
+
-#include <WinStd/Common.h>
+#include <WinStd/Common.h>
-
+
-#include <Windows.h>
+#include <Windows.h>
-#include <WinCrypt.h> // Must include after <Windows.h>
+#include <WinCrypt.h> // Must include after <Windows.h>
-
+
-#include <sal.h>
+#include <sal.h>
-
+
-namespace eap
+namespace eap
-{
+{
-    ///
+    ///
-    /// TLS configuration
+    /// TLS configuration
-    ///
+    ///
-    class config_method_tls;
+    class config_method_tls;
-
+
-    ///
+    ///
-    /// Helper function to compile human-readable certificate name for UI display
+    /// Helper function to compile human-readable certificate name for UI display
-    ///
+    ///
-    winstd::tstring get_cert_title(PCCERT_CONTEXT cert);
+    winstd::tstring get_cert_title(PCCERT_CONTEXT cert);
-}
+}
-
+
-#pragma once
+#pragma once
-
+
-#include "Credentials.h"
+#include "Credentials.h"
-#include "Method.h"
+#include "Method.h"
-#include "TLS.h"
+#include "TLS.h"
-
+
-#include "../../EAPBase/include/Config.h"
+#include "../../EAPBase/include/Config.h"
-
+
-#include <WinStd/Crypt.h>
+#include <WinStd/Crypt.h>
-
+
-#include <Windows.h>
+#include <Windows.h>
-
+
-#include <list>
+#include <list>
-#include <string>
+#include <string>
-
+
-
+
-namespace eap
+namespace eap
-{
+{
-    class config_method_tls : public config_method_with_cred
+    class config_method_tls : public config_method_with_cred
-    {
+    {
-    public:
+    public:
-        ///
+        ///
-        /// Constructs configuration
+        /// Constructs configuration
-        ///
+        ///
-        /// \param[in] mod  EAP module to use for global services
+        /// \param[in] mod  EAP module to use for global services
-        ///
+        ///
-        config_method_tls(_In_ module &mod);
+        config_method_tls(_In_ module &mod);
-
+
-        ///
+        ///
-        /// Copies configuration
+        /// Copies configuration
-        ///
+        ///
-        /// \param[in] other  Configuration to copy from
+        /// \param[in] other  Configuration to copy from
-        ///
+        ///
-        config_method_tls(_In_ const config_method_tls &other);
+        config_method_tls(_In_ const config_method_tls &other);
-
+
-        ///
+        ///
-        /// Moves configuration
+        /// Moves configuration
-        ///
+        ///
-        /// \param[in] other  Configuration to move from
+        /// \param[in] other  Configuration to move from
-        ///
+        ///
-        config_method_tls(_Inout_ config_method_tls &&other);
+        config_method_tls(_Inout_ config_method_tls &&other);
-
+
-        ///
+        ///
-        /// Copies configuration
+        /// Copies configuration
-        ///
+        ///
-        /// \param[in] other  Configuration to copy from
+        /// \param[in] other  Configuration to copy from
-        ///
+        ///
-        /// \returns Reference to this object
+        /// \returns Reference to this object
-        ///
+        ///
-        config_method_tls& operator=(_In_ const config_method_tls &other);
+        config_method_tls& operator=(_In_ const config_method_tls &other);
-
+
-        ///
+        ///
-        /// Moves configuration
+        /// Moves configuration
-        ///
+        ///
-        /// \param[in] other  Configuration to move from
+        /// \param[in] other  Configuration to move from
-        ///
+        ///
-        /// \returns Reference to this object
+        /// \returns Reference to this object
-        ///
+        ///
-        config_method_tls& operator=(_Inout_ config_method_tls &&other);
+        config_method_tls& operator=(_Inout_ config_method_tls &&other);
-
+
-        ///
+        ///
-        /// Clones configuration
+        /// Clones configuration
-        ///
+        ///
-        /// \returns Pointer to cloned configuration
+        /// \returns Pointer to cloned configuration
-        ///
+        ///
-        virtual config* clone() const;
+        virtual config* clone() const;
-
+
-        /// \name XML configuration management
+        /// \name XML configuration management
-        /// @{
+        /// @{
-
+
-        ///
+        ///
-        /// Save to XML document
+        /// Save to XML document
-        ///
+        ///
-        /// \param[in]  pDoc         XML document
+        /// \param[in]  pDoc         XML document
-        /// \param[in]  pConfigRoot  Suggested root element for saving
+        /// \param[in]  pConfigRoot  Suggested root element for saving
-        ///
+        ///
-        virtual void save(_In_ IXMLDOMDocument *pDoc, _In_ IXMLDOMNode *pConfigRoot) const;
+        virtual void save(_In_ IXMLDOMDocument *pDoc, _In_ IXMLDOMNode *pConfigRoot) const;
-
+
-        ///
+        ///
-        /// Load from XML document
+        /// Load from XML document
-        ///
+        ///
-        /// \param[in]  pConfigRoot  Root element for loading
+        /// \param[in]  pConfigRoot  Root element for loading
-        ///
+        ///
-        virtual void load(_In_ IXMLDOMNode *pConfigRoot);
+        virtual void load(_In_ IXMLDOMNode *pConfigRoot);
-
+
-        /// @}
+        /// @}
-
+
-        /// \name BLOB management
+        /// \name BLOB management
-        /// @{
+        /// @{
-
+
-        ///
+        ///
-        /// Packs a configuration
+        /// Packs a configuration
-        ///
+        ///
-        /// \param[inout] cursor  Memory cursor
+        /// \param[inout] cursor  Memory cursor
-        ///
+        ///
-        virtual void operator<<(_Inout_ cursor_out &cursor) const;
+        virtual void operator<<(_Inout_ cursor_out &cursor) const;
-
+
-        ///
+        ///
-        /// Returns packed size of a configuration
+        /// Returns packed size of a configuration
-        ///
+        ///
-        /// \returns Size of data when packed (in bytes)
+        /// \returns Size of data when packed (in bytes)
-        ///
+        ///
-        virtual size_t get_pk_size() const;
+        virtual size_t get_pk_size() const;
-
+
-        ///
+        ///
-        /// Unpacks a configuration
+        /// Unpacks a configuration
-        ///
+        ///
-        /// \param[inout] cursor  Memory cursor
+        /// \param[inout] cursor  Memory cursor
-        ///
+        ///
-        virtual void operator>>(_Inout_ cursor_in &cursor);
+        virtual void operator>>(_Inout_ cursor_in &cursor);
-
+
-        /// @}
+        /// @}
-
+
-        ///
+        ///
-        /// Returns EAP method type of this configuration
+        /// Returns EAP method type of this configuration
-        ///
+        ///
-        /// \returns `eap::type_tls`
+        /// \returns `eap::type_tls`
-        ///
+        ///
-        virtual winstd::eap_type_t get_method_id() const;
+        virtual winstd::eap_type_t get_method_id() const;
-
+
-        ///
+        ///
-        /// Adds CA to the list of trusted root CA's
+        /// Adds CA to the list of trusted root CA's
-        ///
+        ///
-        /// \sa [CertCreateCertificateContext function](https://msdn.microsoft.com/en-us/library/windows/desktop/aa376033.aspx)
+        /// \sa [CertCreateCertificateContext function](https://msdn.microsoft.com/en-us/library/windows/desktop/aa376033.aspx)
-        ///
+        ///
-        bool add_trusted_ca(_In_ DWORD dwCertEncodingType, _In_ const BYTE *pbCertEncoded, _In_ DWORD cbCertEncoded);
+        bool add_trusted_ca(_In_ DWORD dwCertEncodingType, _In_ const BYTE *pbCertEncoded, _In_ DWORD cbCertEncoded);
-
+
-    public:
+    public:
-        std::list<winstd::cert_context> m_trusted_root_ca;  ///< Trusted root CAs
+        std::list<winstd::cert_context> m_trusted_root_ca;  ///< Trusted root CAs
-        std::list<std::string> m_server_names;              ///< Acceptable authenticating server names
+        std::list<std::wstring> m_server_names;             ///< Acceptable authenticating server names
-
+
-        // Following members are used for session resumptions. They are not exported/imported to XML.
+        // Following members are used for session resumptions. They are not exported/imported to XML.
-        sanitizing_blob m_session_id;                       ///< TLS session ID
+        sanitizing_blob m_session_id;                       ///< TLS session ID
-        tls_master_secret m_master_secret;                  ///< TLS master secret
+        tls_master_secret m_master_secret;                  ///< TLS master secret
-    };
+    };
-}
+}

lib/TLS/src/Config.cpp

@@ -1,305 +1,298 @@
-/*
+/*
-    Copyright 2015-2016 Amebis
+    Copyright 2015-2016 Amebis
-    Copyright 2016 GÉANT
+    Copyright 2016 GÉANT
-
+
-    This file is part of GÉANTLink.
+    This file is part of GÉANTLink.
-
+
-    GÉANTLink is free software: you can redistribute it and/or modify it
+    GÉANTLink is free software: you can redistribute it and/or modify it
-    under the terms of the GNU General Public License as published by
+    under the terms of the GNU General Public License as published by
-    the Free Software Foundation, either version 3 of the License, or
+    the Free Software Foundation, either version 3 of the License, or
-    (at your option) any later version.
+    (at your option) any later version.
-
+
-    GÉANTLink is distributed in the hope that it will be useful, but
+    GÉANTLink is distributed in the hope that it will be useful, but
-    WITHOUT ANY WARRANTY; without even the implied warranty of
+    WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
+    GNU General Public License for more details.
-
+
-    You should have received a copy of the GNU General Public License
+    You should have received a copy of the GNU General Public License
-    along with GÉANTLink. If not, see <http://www.gnu.org/licenses/>.
+    along with GÉANTLink. If not, see <http://www.gnu.org/licenses/>.
-*/
+*/
-
+
-#include "StdAfx.h"
+#include "StdAfx.h"
-
+
-#pragma comment(lib, "Cryptui.lib")
+#pragma comment(lib, "Cryptui.lib")
-
+
-using namespace std;
+using namespace std;
-using namespace winstd;
+using namespace winstd;
-
+
-
+
-//////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////
-// eap::get_cert_title
+// eap::get_cert_title
-//////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////
-
+
-tstring eap::get_cert_title(PCCERT_CONTEXT cert)
+tstring eap::get_cert_title(PCCERT_CONTEXT cert)
-{
+{
-    tstring name, str, issuer, title;
+    tstring name, str, issuer, title;
-    FILETIME ft;
+    FILETIME ft;
-    SYSTEMTIME st;
+    SYSTEMTIME st;
-
+
-    // Prepare certificate information
+    // Prepare certificate information
-    CertGetNameString(cert, CERT_NAME_SIMPLE_DISPLAY_TYPE, 0, NULL, name);
+    CertGetNameString(cert, CERT_NAME_SIMPLE_DISPLAY_TYPE, 0, NULL, name);
-    title += name;
+    title += name;
-
+
-    FileTimeToLocalFileTime(&(cert->pCertInfo->NotBefore), &ft);
+    FileTimeToLocalFileTime(&(cert->pCertInfo->NotBefore), &ft);
-    FileTimeToSystemTime(&ft, &st);
+    FileTimeToSystemTime(&ft, &st);
-    GetDateFormat(LOCALE_USER_DEFAULT, DATE_SHORTDATE, &st, NULL, str);
+    GetDateFormat(LOCALE_USER_DEFAULT, DATE_SHORTDATE, &st, NULL, str);
-    title += _T(", ");
+    title += _T(", ");
-    title += str;
+    title += str;
-
+
-    FileTimeToLocalFileTime(&(cert->pCertInfo->NotAfter ), &ft);
+    FileTimeToLocalFileTime(&(cert->pCertInfo->NotAfter ), &ft);
-    FileTimeToSystemTime(&ft, &st);
+    FileTimeToSystemTime(&ft, &st);
-    GetDateFormat(LOCALE_USER_DEFAULT, DATE_SHORTDATE, &st, NULL, str);
+    GetDateFormat(LOCALE_USER_DEFAULT, DATE_SHORTDATE, &st, NULL, str);
-    title += _T('-');
+    title += _T('-');
-    title += str;
+    title += str;
-
+
-    CertGetNameString(cert, CERT_NAME_SIMPLE_DISPLAY_TYPE, CERT_NAME_ISSUER_FLAG, NULL, issuer);
+    CertGetNameString(cert, CERT_NAME_SIMPLE_DISPLAY_TYPE, CERT_NAME_ISSUER_FLAG, NULL, issuer);
-    if (name != issuer) {
+    if (name != issuer) {
-        title += _T(", ");
+        title += _T(", ");
-        title += issuer;
+        title += issuer;
-    }
+    }
-
+
-    return title;
+    return title;
-}
+}
-
+
-
+
-//////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////
-// eap::config_method_tls
+// eap::config_method_tls
-//////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////
-
+
-eap::config_method_tls::config_method_tls(_In_ module &mod) : config_method_with_cred(mod)
+eap::config_method_tls::config_method_tls(_In_ module &mod) : config_method_with_cred(mod)
-{
+{
-    m_preshared.reset(new credentials_tls(mod));
+    m_preshared.reset(new credentials_tls(mod));
-}
+}
-
+
-
+
-eap::config_method_tls::config_method_tls(_In_ const config_method_tls &other) :
+eap::config_method_tls::config_method_tls(_In_ const config_method_tls &other) :
-    m_trusted_root_ca(other.m_trusted_root_ca),
+    m_trusted_root_ca(other.m_trusted_root_ca),
-    m_server_names(other.m_server_names),
+    m_server_names(other.m_server_names),
-    m_session_id(other.m_session_id),
+    m_session_id(other.m_session_id),
-    m_master_secret(other.m_master_secret),
+    m_master_secret(other.m_master_secret),
-    config_method_with_cred(other)
+    config_method_with_cred(other)
-{
+{
-}
+}
-
+
-
+
-eap::config_method_tls::config_method_tls(_Inout_ config_method_tls &&other) :
+eap::config_method_tls::config_method_tls(_Inout_ config_method_tls &&other) :
-    m_trusted_root_ca(std::move(other.m_trusted_root_ca)),
+    m_trusted_root_ca(std::move(other.m_trusted_root_ca)),
-    m_server_names(std::move(other.m_server_names)),
+    m_server_names(std::move(other.m_server_names)),
-    m_session_id(std::move(other.m_session_id)),
+    m_session_id(std::move(other.m_session_id)),
-    m_master_secret(std::move(other.m_master_secret)),
+    m_master_secret(std::move(other.m_master_secret)),
-    config_method_with_cred(std::move(other))
+    config_method_with_cred(std::move(other))
-{
+{
-}
+}
-
+
-
+
-eap::config_method_tls& eap::config_method_tls::operator=(_In_ const config_method_tls &other)
+eap::config_method_tls& eap::config_method_tls::operator=(_In_ const config_method_tls &other)
-{
+{
-    if (this != &other) {
+    if (this != &other) {
-        (config_method_with_cred&)*this = other;
+        (config_method_with_cred&)*this = other;
-        m_trusted_root_ca = other.m_trusted_root_ca;
+        m_trusted_root_ca = other.m_trusted_root_ca;
-        m_server_names    = other.m_server_names;
+        m_server_names    = other.m_server_names;
-        m_session_id      = other.m_session_id;
+        m_session_id      = other.m_session_id;
-        m_master_secret   = other.m_master_secret;
+        m_master_secret   = other.m_master_secret;
-    }
+    }
-
+
-    return *this;
+    return *this;
-}
+}
-
+
-
+
-eap::config_method_tls& eap::config_method_tls::operator=(_Inout_ config_method_tls &&other)
+eap::config_method_tls& eap::config_method_tls::operator=(_Inout_ config_method_tls &&other)
-{
+{
-    if (this != &other) {
+    if (this != &other) {
-        (config_method_with_cred&&)*this = std::move(other);
+        (config_method_with_cred&&)*this = std::move(other);
-        m_trusted_root_ca = std::move(other.m_trusted_root_ca);
+        m_trusted_root_ca = std::move(other.m_trusted_root_ca);
-        m_server_names    = std::move(other.m_server_names);
+        m_server_names    = std::move(other.m_server_names);
-        m_session_id      = std::move(other.m_session_id);
+        m_session_id      = std::move(other.m_session_id);
-        m_master_secret   = std::move(other.m_master_secret);
+        m_master_secret   = std::move(other.m_master_secret);
-    }
+    }
-
+
-    return *this;
+    return *this;
-}
+}
-
+
-
+
-eap::config* eap::config_method_tls::clone() const
+eap::config* eap::config_method_tls::clone() const
-{
+{
-    return new config_method_tls(*this);
+    return new config_method_tls(*this);
-}
+}
-
+
-
+
-void eap::config_method_tls::save(_In_ IXMLDOMDocument *pDoc, _In_ IXMLDOMNode *pConfigRoot) const
+void eap::config_method_tls::save(_In_ IXMLDOMDocument *pDoc, _In_ IXMLDOMNode *pConfigRoot) const
-{
+{
-    assert(pDoc);
+    assert(pDoc);
-    assert(pConfigRoot);
+    assert(pConfigRoot);
-
+
-    config_method_with_cred::save(pDoc, pConfigRoot);
+    config_method_with_cred::save(pDoc, pConfigRoot);
-
+
-    const bstr bstrNamespace(L"urn:ietf:params:xml:ns:yang:ietf-eap-metadata");
+    const bstr bstrNamespace(L"urn:ietf:params:xml:ns:yang:ietf-eap-metadata");
-    HRESULT hr;
+    HRESULT hr;
-
+
-    // <ServerSideCredential>
+    // <ServerSideCredential>
-    com_obj<IXMLDOMElement> pXmlElServerSideCredential;
+    com_obj<IXMLDOMElement> pXmlElServerSideCredential;
-    if (FAILED(hr = eapxml::create_element(pDoc, pConfigRoot, bstr(L"eap-metadata:ServerSideCredential"), bstr(L"ServerSideCredential"), bstrNamespace, &pXmlElServerSideCredential)))
+    if (FAILED(hr = eapxml::create_element(pDoc, pConfigRoot, bstr(L"eap-metadata:ServerSideCredential"), bstr(L"ServerSideCredential"), bstrNamespace, &pXmlElServerSideCredential)))
-        throw com_runtime_error(hr, __FUNCTION__ " Error creating <ServerSideCredential> element.");
+        throw com_runtime_error(hr, __FUNCTION__ " Error creating <ServerSideCredential> element.");
-
+
-    for (list<cert_context>::const_iterator i = m_trusted_root_ca.begin(), i_end = m_trusted_root_ca.end(); i != i_end; ++i) {
+    for (list<cert_context>::const_iterator i = m_trusted_root_ca.begin(), i_end = m_trusted_root_ca.end(); i != i_end; ++i) {
-        // <CA>
+        // <CA>
-        com_obj<IXMLDOMElement> pXmlElCA;
+        com_obj<IXMLDOMElement> pXmlElCA;
-        if (FAILED(hr = eapxml::create_element(pDoc, bstr(L"CA"), bstrNamespace, &pXmlElCA)))
+        if (FAILED(hr = eapxml::create_element(pDoc, bstr(L"CA"), bstrNamespace, &pXmlElCA)))
-            throw com_runtime_error(hr, __FUNCTION__ " Error creating <CA> element.");
+            throw com_runtime_error(hr, __FUNCTION__ " Error creating <CA> element.");
-
+
-        // <CA>/<format>
+        // <CA>/<format>
-        if (FAILED(hr = eapxml::put_element_value(pDoc, pXmlElCA, bstr(L"format"), bstrNamespace, bstr(L"PEM"))))
+        if (FAILED(hr = eapxml::put_element_value(pDoc, pXmlElCA, bstr(L"format"), bstrNamespace, bstr(L"PEM"))))
-            throw com_runtime_error(hr, __FUNCTION__ " Error creating <format> element.");
+            throw com_runtime_error(hr, __FUNCTION__ " Error creating <format> element.");
-
+
-        // <CA>/<cert-data>
+        // <CA>/<cert-data>
-        const cert_context &cc = *i;
+        const cert_context &cc = *i;
-        if (FAILED(hr = eapxml::put_element_base64(pDoc, pXmlElCA, bstr(L"cert-data"), bstrNamespace, cc->pbCertEncoded, cc->cbCertEncoded)))
+        if (FAILED(hr = eapxml::put_element_base64(pDoc, pXmlElCA, bstr(L"cert-data"), bstrNamespace, cc->pbCertEncoded, cc->cbCertEncoded)))
-            throw com_runtime_error(hr, __FUNCTION__ " Error creating <cert-data> element.");
+            throw com_runtime_error(hr, __FUNCTION__ " Error creating <cert-data> element.");
-
+
-        if (FAILED(hr = pXmlElServerSideCredential->appendChild(pXmlElCA, NULL)))
+        if (FAILED(hr = pXmlElServerSideCredential->appendChild(pXmlElCA, NULL)))
-            throw com_runtime_error(hr, __FUNCTION__ " Error appending <CA> element.");
+            throw com_runtime_error(hr, __FUNCTION__ " Error appending <CA> element.");
-    }
+    }
-
+
-    // <ServerName>
+    // <ServerName>
-    for (list<string>::const_iterator i = m_server_names.begin(), i_end = m_server_names.end(); i != i_end; ++i) {
+    for (list<wstring>::const_iterator i = m_server_names.begin(), i_end = m_server_names.end(); i != i_end; ++i) {
-        wstring str;
+        if (FAILED(hr = eapxml::put_element_value(pDoc, pXmlElServerSideCredential, bstr(L"ServerName"), bstrNamespace, bstr(*i))))
-        MultiByteToWideChar(CP_UTF8, 0, i->c_str(), (int)i->length(), str);
+            throw com_runtime_error(hr, __FUNCTION__ " Error creating <ServerName> element.");
-        if (FAILED(hr = eapxml::put_element_value(pDoc, pXmlElServerSideCredential, bstr(L"ServerName"), bstrNamespace, bstr(str))))
+    }
-            throw com_runtime_error(hr, __FUNCTION__ " Error creating <ServerName> element.");
+}
-    }
+
-}
+
-
+void eap::config_method_tls::load(_In_ IXMLDOMNode *pConfigRoot)
-
+{
-void eap::config_method_tls::load(_In_ IXMLDOMNode *pConfigRoot)
+    assert(pConfigRoot);
-{
+
-    assert(pConfigRoot);
+    config_method_with_cred::load(pConfigRoot);
-
+
-    config_method_with_cred::load(pConfigRoot);
+    std::wstring xpath(eapxml::get_xpath(pConfigRoot));
-
+
-    std::wstring xpath(eapxml::get_xpath(pConfigRoot));
+    m_trusted_root_ca.clear();
-
+    m_server_names.clear();
-    m_trusted_root_ca.clear();
+
-    m_server_names.clear();
+    // <ServerSideCredential>
-
+    com_obj<IXMLDOMElement> pXmlElServerSideCredential;
-    // <ServerSideCredential>
+    if (SUCCEEDED(eapxml::select_element(pConfigRoot, bstr(L"eap-metadata:ServerSideCredential"), &pXmlElServerSideCredential))) {
-    com_obj<IXMLDOMElement> pXmlElServerSideCredential;
+        std::wstring xpathServerSideCredential(xpath + L"/ServerSideCredential");
-    if (SUCCEEDED(eapxml::select_element(pConfigRoot, bstr(L"eap-metadata:ServerSideCredential"), &pXmlElServerSideCredential))) {
+
-        std::wstring xpathServerSideCredential(xpath + L"/ServerSideCredential");
+        // <CA>
-
+        com_obj<IXMLDOMNodeList> pXmlListCAs;
-        // <CA>
+        long lCACount = 0;
-        com_obj<IXMLDOMNodeList> pXmlListCAs;
+        if (SUCCEEDED(eapxml::select_nodes(pXmlElServerSideCredential, bstr(L"eap-metadata:CA"), &pXmlListCAs)) && SUCCEEDED(pXmlListCAs->get_length(&lCACount))) {
-        long lCACount = 0;
+            for (long j = 0; j < lCACount; j++) {
-        if (SUCCEEDED(eapxml::select_nodes(pXmlElServerSideCredential, bstr(L"eap-metadata:CA"), &pXmlListCAs)) && SUCCEEDED(pXmlListCAs->get_length(&lCACount))) {
+                // Load CA certificate.
-            for (long j = 0; j < lCACount; j++) {
+                com_obj<IXMLDOMNode> pXmlElCA;
-                // Load CA certificate.
+                pXmlListCAs->get_item(j, &pXmlElCA);
-                com_obj<IXMLDOMNode> pXmlElCA;
+                bstr bstrFormat;
-                pXmlListCAs->get_item(j, &pXmlElCA);
+                if (FAILED(eapxml::get_element_value(pXmlElCA, bstr(L"eap-metadata:format"), &bstrFormat))) {
-                bstr bstrFormat;
+                    // <format> not specified.
-                if (FAILED(eapxml::get_element_value(pXmlElCA, bstr(L"eap-metadata:format"), &bstrFormat))) {
+                    continue;
-                    // <format> not specified.
+                }
-                    continue;
+
-                }
+                if (CompareStringEx(LOCALE_NAME_INVARIANT, NORM_IGNORECASE, bstrFormat, bstrFormat.length(), L"PEM", -1, NULL, NULL, 0) != CSTR_EQUAL) {
-
+                    // Certificate must be PEM encoded.
-                if (CompareStringEx(LOCALE_NAME_INVARIANT, NORM_IGNORECASE, bstrFormat, bstrFormat.length(), L"PEM", -1, NULL, NULL, 0) != CSTR_EQUAL) {
+                    continue;
-                    // Certificate must be PEM encoded.
+                }
-                    continue;
+
-                }
+                vector<unsigned char> aData;
-
+                if (FAILED(eapxml::get_element_base64(pXmlElCA, bstr(L"eap-metadata:cert-data"), aData))) {
-                vector<unsigned char> aData;
+                    // Error reading <cert-data> element.
-                if (FAILED(eapxml::get_element_base64(pXmlElCA, bstr(L"eap-metadata:cert-data"), aData))) {
+                    continue;
-                    // Error reading <cert-data> element.
+                }
-                    continue;
+
-                }
+                add_trusted_ca(X509_ASN_ENCODING | PKCS_7_ASN_ENCODING, aData.data(), (DWORD)aData.size());
-
+            }
-                add_trusted_ca(X509_ASN_ENCODING | PKCS_7_ASN_ENCODING, aData.data(), (DWORD)aData.size());
+
-            }
+            // Log loaded CA certificates.
-
+            list<tstring> cert_names;
-            // Log loaded CA certificates.
+            for (std::list<winstd::cert_context>::const_iterator cert = m_trusted_root_ca.cbegin(), cert_end = m_trusted_root_ca.cend(); cert != cert_end; ++cert)
-            list<tstring> cert_names;
+                cert_names.push_back(std::move(get_cert_title(*cert)));
-            for (std::list<winstd::cert_context>::const_iterator cert = m_trusted_root_ca.cbegin(), cert_end = m_trusted_root_ca.cend(); cert != cert_end; ++cert)
+            m_module.log_config((xpathServerSideCredential + L"/CA").c_str(), cert_names);
-                cert_names.push_back(std::move(get_cert_title(*cert)));
+        }
-            m_module.log_config((xpathServerSideCredential + L"/CA").c_str(), cert_names);
+
-        }
+        // <ServerName>
-
+        com_obj<IXMLDOMNodeList> pXmlListServerIDs;
-        // <ServerName>
+        long lServerIDCount = 0;
-        com_obj<IXMLDOMNodeList> pXmlListServerIDs;
+        if (SUCCEEDED(eapxml::select_nodes(pXmlElServerSideCredential, bstr(L"eap-metadata:ServerName"), &pXmlListServerIDs)) && SUCCEEDED(pXmlListServerIDs->get_length(&lServerIDCount))) {
-        long lServerIDCount = 0;
+            for (long j = 0; j < lServerIDCount; j++) {
-        if (SUCCEEDED(eapxml::select_nodes(pXmlElServerSideCredential, bstr(L"eap-metadata:ServerName"), &pXmlListServerIDs)) && SUCCEEDED(pXmlListServerIDs->get_length(&lServerIDCount))) {
+                // Load server name (<ServerName>).
-            for (long j = 0; j < lServerIDCount; j++) {
+                com_obj<IXMLDOMNode> pXmlElServerID;
-                // Load server name (<ServerName>).
+                pXmlListServerIDs->get_item(j, &pXmlElServerID);
-                com_obj<IXMLDOMNode> pXmlElServerID;
+                bstr bstrServerID;
-                pXmlListServerIDs->get_item(j, &pXmlElServerID);
+                pXmlElServerID->get_text(&bstrServerID);
-                bstr bstrServerID;
+                m_server_names.push_back(wstring(bstrServerID));
-                pXmlElServerID->get_text(&bstrServerID);
+            }
-
+
-                // Server names (FQDNs) are always ASCII. Hopefully. Convert them to UTF-8 anyway for consistent comparison. CP_ANSI varies.
+            m_module.log_config((xpathServerSideCredential + L"/ServerName").c_str(), m_server_names);
-                string str;
+        }
-                WideCharToMultiByte(CP_UTF8, 0, bstrServerID, bstrServerID.length(), str, NULL, NULL);
+    }
-
+}
-                m_server_names.push_back(str);
+
-            }
+
-
+void eap::config_method_tls::operator<<(_Inout_ cursor_out &cursor) const
-            m_module.log_config((xpathServerSideCredential + L"/ServerName").c_str(), m_server_names);
+{
-        }
+    config_method_with_cred::operator<<(cursor);
-    }
+    cursor << m_trusted_root_ca;
-}
+    cursor << m_server_names   ;
-
+    cursor << m_session_id     ;
-
+    cursor << m_master_secret  ;
-void eap::config_method_tls::operator<<(_Inout_ cursor_out &cursor) const
+}
-{
+
-    config_method_with_cred::operator<<(cursor);
+
-    cursor << m_trusted_root_ca;
+size_t eap::config_method_tls::get_pk_size() const
-    cursor << m_server_names   ;
+{
-    cursor << m_session_id     ;
+    return
-    cursor << m_master_secret  ;
+        config_method_with_cred::get_pk_size() +
-}
+        pksizeof(m_trusted_root_ca) +
-
+        pksizeof(m_server_names   ) +
-
+        pksizeof(m_session_id     ) +
-size_t eap::config_method_tls::get_pk_size() const
+        pksizeof(m_master_secret  );
-{
+}
-    return
+
-        config_method_with_cred::get_pk_size() +
+
-        pksizeof(m_trusted_root_ca) +
+void eap::config_method_tls::operator>>(_Inout_ cursor_in &cursor)
-        pksizeof(m_server_names   ) +
+{
-        pksizeof(m_session_id     ) +
+    config_method_with_cred::operator>>(cursor);
-        pksizeof(m_master_secret  );
+    cursor >> m_trusted_root_ca;
-}
+    cursor >> m_server_names   ;
-
+    cursor >> m_session_id     ;
-
+    cursor >> m_master_secret  ;
-void eap::config_method_tls::operator>>(_Inout_ cursor_in &cursor)
+}
-{
+
-    config_method_with_cred::operator>>(cursor);
+
-    cursor >> m_trusted_root_ca;
+eap_type_t eap::config_method_tls::get_method_id() const
-    cursor >> m_server_names   ;
+{
-    cursor >> m_session_id     ;
+    return eap_type_tls;
-    cursor >> m_master_secret  ;
+}
-}
+
-
+
-
+bool eap::config_method_tls::add_trusted_ca(_In_  DWORD dwCertEncodingType, _In_  const BYTE *pbCertEncoded, _In_  DWORD cbCertEncoded)
-eap_type_t eap::config_method_tls::get_method_id() const
+{
-{
+    cert_context cert;
-    return eap_type_tls;
+    if (!cert.create(dwCertEncodingType, pbCertEncoded, cbCertEncoded)) {
-}
+        // Invalid or unsupported certificate.
-
+        return false;
-
+    }
-bool eap::config_method_tls::add_trusted_ca(_In_  DWORD dwCertEncodingType, _In_  const BYTE *pbCertEncoded, _In_  DWORD cbCertEncoded)
+
-{
+    for (list<cert_context>::const_iterator i = m_trusted_root_ca.cbegin(), i_end = m_trusted_root_ca.cend();; ++i) {
-    cert_context cert;
+        if (i != i_end) {
-    if (!cert.create(dwCertEncodingType, pbCertEncoded, cbCertEncoded)) {
+            if (*i == cert) {
-        // Invalid or unsupported certificate.
+                // This certificate is already on the list.
-        return false;
+                return false;
-    }
+            }
-
+        } else {
-    for (list<cert_context>::const_iterator i = m_trusted_root_ca.cbegin(), i_end = m_trusted_root_ca.cend();; ++i) {
+            // End of list reached. Append certificate.
-        if (i != i_end) {
+            m_trusted_root_ca.push_back(std::move(cert));
-            if (*i == cert) {
+            return true;
-                // This certificate is already on the list.
+        }
-                return false;
+    }
-            }
+}
-        } else {
-            // End of list reached. Append certificate.
-            m_trusted_root_ca.push_back(std::move(cert));
-            return true;
-        }
-    }
-}

lib/TLS_UI/include/TLS_UI.h

@@ -1,342 +1,342 @@
-/*
+/*
-    Copyright 2015-2016 Amebis
+    Copyright 2015-2016 Amebis
-    Copyright 2016 GÉANT
+    Copyright 2016 GÉANT
-
+
-    This file is part of GÉANTLink.
+    This file is part of GÉANTLink.
-
+
-    GÉANTLink is free software: you can redistribute it and/or modify it
+    GÉANTLink is free software: you can redistribute it and/or modify it
-    under the terms of the GNU General Public License as published by
+    under the terms of the GNU General Public License as published by
-    the Free Software Foundation, either version 3 of the License, or
+    the Free Software Foundation, either version 3 of the License, or
-    (at your option) any later version.
+    (at your option) any later version.
-
+
-    GÉANTLink is distributed in the hope that it will be useful, but
+    GÉANTLink is distributed in the hope that it will be useful, but
-    WITHOUT ANY WARRANTY; without even the implied warranty of
+    WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
+    GNU General Public License for more details.
-
+
-    You should have received a copy of the GNU General Public License
+    You should have received a copy of the GNU General Public License
-    along with GÉANTLink. If not, see <http://www.gnu.org/licenses/>.
+    along with GÉANTLink. If not, see <http://www.gnu.org/licenses/>.
-*/
+*/
-
+
-#include "../../EAPBase_UI/include/EAP_UI.h"
+#include "../../EAPBase_UI/include/EAP_UI.h"
-#include "../../TLS/include/Config.h"
+#include "../../TLS/include/Config.h"
-#include "../../TLS/include/Credentials.h"
+#include "../../TLS/include/Credentials.h"
-
+
-#include <WinStd/Common.h>
+#include <WinStd/Common.h>
-
+
-#include <wx/filedlg.h>
+#include <wx/filedlg.h>
-#include <wx/msgdlg.h>
+#include <wx/msgdlg.h>
-
+
-#include <Windows.h>
+#include <Windows.h>
-#include <cryptuiapi.h>
+#include <cryptuiapi.h>
-#include <WinCrypt.h> // Must include after <Windows.h>
+#include <WinCrypt.h> // Must include after <Windows.h>
-
+
-#include <list>
+#include <list>
-#include <string>
+#include <string>
-
+
-
+
-///
+///
-/// Helper class for auto-destroyable certificates used in wxWidget's item containers
+/// Helper class for auto-destroyable certificates used in wxWidget's item containers
-///
+///
-class wxCertificateClientData;
+class wxCertificateClientData;
-
+
-///
+///
-/// Validator for host name
+/// Validator for host name
-///
+///
-class wxHostNameValidator;
+class wxHostNameValidator;
-
+
-///
+///
-/// Validator for FQDN
+/// Validator for FQDN
-///
+///
-class wxFQDNValidator;
+class wxFQDNValidator;
-
+
-///
+///
-/// Validator for FQDN lists
+/// Validator for FQDN lists
-///
+///
-class wxFQDNListValidator;
+class wxFQDNListValidator;
-
+
-///
+///
-/// TLS credential panel
+/// TLS credential panel
-///
+///
-class wxTLSCredentialsPanel;
+class wxTLSCredentialsPanel;
-
+
-///
+///
-/// TLS server trust configuration panel
+/// TLS server trust configuration panel
-///
+///
-class wxTLSServerTrustPanel;
+class wxTLSServerTrustPanel;
-
+
-///
+///
-/// TLS credentials configuration panel
+/// TLS credentials configuration panel
-///
+///
-typedef wxEAPCredentialsConfigPanel<eap::credentials_tls, wxTLSCredentialsPanel> wxTLSCredentialsConfigPanel;
+typedef wxEAPCredentialsConfigPanel<eap::credentials_tls, wxTLSCredentialsPanel> wxTLSCredentialsConfigPanel;
-
+
-///
+///
-/// TLS configuration panel
+/// TLS configuration panel
-///
+///
-class wxTLSConfigPanel;
+class wxTLSConfigPanel;
-
+
-#pragma once
+#pragma once
-
+
-#include "../res/wxTLS_UI.h"
+#include "../res/wxTLS_UI.h"
-
+
-#include <WinStd/Win.h>
+#include <WinStd/Win.h>
-
+
-#include <wx/clntdata.h>
+#include <wx/clntdata.h>
-#include <wx/icon.h>
+#include <wx/icon.h>
-#include <wx/panel.h>
+#include <wx/panel.h>
-#include <wx/textctrl.h>
+#include <wx/textctrl.h>
-#include <wx/validate.h>
+#include <wx/validate.h>
-
+
-#include <list>
+#include <list>
-#include <string>
+#include <string>
-#include <vector>
+#include <vector>
-
+
-
+
-class wxCertificateClientData : public wxClientData
+class wxCertificateClientData : public wxClientData
-{
+{
-public:
+public:
-    ///
+    ///
-    /// Constructs client data object with existing handle
+    /// Constructs client data object with existing handle
-    ///
+    ///
-    wxCertificateClientData(PCCERT_CONTEXT cert);
+    wxCertificateClientData(PCCERT_CONTEXT cert);
-
+
-    ///
+    ///
-    /// Releases certificate handle and destructs the object
+    /// Releases certificate handle and destructs the object
-    ///
+    ///
-    virtual ~wxCertificateClientData();
+    virtual ~wxCertificateClientData();
-
+
-public:
+public:
-    PCCERT_CONTEXT m_cert;  ///< Certificate
+    PCCERT_CONTEXT m_cert;  ///< Certificate
-};
+};
-
+
-
+
-class wxHostNameValidator : public wxValidator
+class wxHostNameValidator : public wxValidator
-{
+{
-    wxDECLARE_DYNAMIC_CLASS(wxHostNameValidator);
+    wxDECLARE_DYNAMIC_CLASS(wxHostNameValidator);
-    wxDECLARE_NO_ASSIGN_CLASS(wxHostNameValidator);
+    wxDECLARE_NO_ASSIGN_CLASS(wxHostNameValidator);
-
+
-public:
+public:
-    ///
+    ///
-    /// Construct the validator with a value to store data
+    /// Construct the validator with a value to store data
-    ///
+    ///
-    wxHostNameValidator(std::string *val = NULL);
+    wxHostNameValidator(std::wstring *val = NULL);
-
+
-    ///
+    ///
-    /// Copy constructor
+    /// Copy constructor
-    ///
+    ///
-    wxHostNameValidator(const wxHostNameValidator &other);
+    wxHostNameValidator(const wxHostNameValidator &other);
-
+
-    ///
+    ///
-    /// Copies this validator
+    /// Copies this validator
-    ///
+    ///
-    virtual wxObject* Clone() const;
+    virtual wxObject* Clone() const;
-
+
-    ///
+    ///
-    /// Validates the value
+    /// Validates the value
-    ///
+    ///
-    virtual bool Validate(wxWindow *parent);
+    virtual bool Validate(wxWindow *parent);
-
+
-    ///
+    ///
-    /// Transfers the value to the window
+    /// Transfers the value to the window
-    ///
+    ///
-    virtual bool TransferToWindow();
+    virtual bool TransferToWindow();
-
+
-    ///
+    ///
-    /// Transfers the value from the window
+    /// Transfers the value from the window
-    ///
+    ///
-    virtual bool TransferFromWindow();
+    virtual bool TransferFromWindow();
-
+
-    ///
+    ///
-    /// Parses FQDN value
+    /// Parses FQDN value
-    ///
+    ///
-    static bool Parse(const wxString &val_in, size_t i_start, size_t i_end, wxTextCtrl *ctrl, wxWindow *parent, std::string *val_out = NULL);
+    static bool Parse(const wxString &val_in, size_t i_start, size_t i_end, wxTextCtrl *ctrl, wxWindow *parent, std::wstring *val_out = NULL);
-
+
-protected:
+protected:
-    std::string *m_val; ///< Pointer to variable to receive control's parsed value
+    std::wstring *m_val; ///< Pointer to variable to receive control's parsed value
-};
+};
-
+
-
+
-class wxFQDNValidator : public wxValidator
+class wxFQDNValidator : public wxValidator
-{
+{
-    wxDECLARE_DYNAMIC_CLASS(wxFQDNValidator);
+    wxDECLARE_DYNAMIC_CLASS(wxFQDNValidator);
-    wxDECLARE_NO_ASSIGN_CLASS(wxFQDNValidator);
+    wxDECLARE_NO_ASSIGN_CLASS(wxFQDNValidator);
-
+
-public:
+public:
-    ///
+    ///
-    /// Construct the validator with a value to store data
+    /// Construct the validator with a value to store data
-    ///
+    ///
-    wxFQDNValidator(std::string *val = NULL);
+    wxFQDNValidator(std::wstring *val = NULL);
-
+
-    ///
+    ///
-    /// Copy constructor
+    /// Copy constructor
-    ///
+    ///
-    wxFQDNValidator(const wxFQDNValidator &other);
+    wxFQDNValidator(const wxFQDNValidator &other);
-
+
-    ///
+    ///
-    /// Copies this validator
+    /// Copies this validator
-    ///
+    ///
-    virtual wxObject* Clone() const;
+    virtual wxObject* Clone() const;
-
+
-    ///
+    ///
-    /// Validates the value
+    /// Validates the value
-    ///
+    ///
-    virtual bool Validate(wxWindow *parent);
+    virtual bool Validate(wxWindow *parent);
-
+
-    ///
+    ///
-    /// Transfers the value to the window
+    /// Transfers the value to the window
-    ///
+    ///
-    virtual bool TransferToWindow();
+    virtual bool TransferToWindow();
-
+
-    ///
+    ///
-    /// Transfers the value from the window
+    /// Transfers the value from the window
-    ///
+    ///
-    virtual bool TransferFromWindow();
+    virtual bool TransferFromWindow();
-
+
-    ///
+    ///
-    /// Parses FQDN value
+    /// Parses FQDN value
-    ///
+    ///
-    static bool Parse(const wxString &val_in, size_t i_start, size_t i_end, wxTextCtrl *ctrl, wxWindow *parent, std::string *val_out = NULL);
+    static bool Parse(const wxString &val_in, size_t i_start, size_t i_end, wxTextCtrl *ctrl, wxWindow *parent, std::wstring *val_out = NULL);
-
+
-protected:
+protected:
-    std::string *m_val; ///< Pointer to variable to receive control's parsed value
+    std::wstring *m_val; ///< Pointer to variable to receive control's parsed value
-};
+};
-
+
-
+
-class wxFQDNListValidator : public wxValidator
+class wxFQDNListValidator : public wxValidator
-{
+{
-    wxDECLARE_DYNAMIC_CLASS(wxFQDNListValidator);
+    wxDECLARE_DYNAMIC_CLASS(wxFQDNListValidator);
-    wxDECLARE_NO_ASSIGN_CLASS(wxFQDNListValidator);
+    wxDECLARE_NO_ASSIGN_CLASS(wxFQDNListValidator);
-
+
-public:
+public:
-    ///
+    ///
-    /// Construct the validator with a value to store data
+    /// Construct the validator with a value to store data
-    ///
+    ///
-    wxFQDNListValidator(std::list<std::string> *val = NULL);
+    wxFQDNListValidator(std::list<std::wstring> *val = NULL);
-
+
-    ///
+    ///
-    /// Copy constructor
+    /// Copy constructor
-    ///
+    ///
-    wxFQDNListValidator(const wxFQDNListValidator &other);
+    wxFQDNListValidator(const wxFQDNListValidator &other);
-
+
-    ///
+    ///
-    /// Copies this validator
+    /// Copies this validator
-    ///
+    ///
-    virtual wxObject* Clone() const;
+    virtual wxObject* Clone() const;
-
+
-    ///
+    ///
-    /// Validates the value
+    /// Validates the value
-    ///
+    ///
-    virtual bool Validate(wxWindow *parent);
+    virtual bool Validate(wxWindow *parent);
-
+
-    ///
+    ///
-    /// Transfers the value to the window
+    /// Transfers the value to the window
-    ///
+    ///
-    virtual bool TransferToWindow();
+    virtual bool TransferToWindow();
-
+
-    ///
+    ///
-    /// Transfers the value from the window
+    /// Transfers the value from the window
-    ///
+    ///
-    virtual bool TransferFromWindow();
+    virtual bool TransferFromWindow();
-
+
-    ///
+    ///
-    /// Parses FQDN list value
+    /// Parses FQDN list value
-    ///
+    ///
-    static bool Parse(const wxString &val_in, size_t i_start, size_t i_end, wxTextCtrl *ctrl, wxWindow *parent, std::list<std::string> *val_out = NULL);
+    static bool Parse(const wxString &val_in, size_t i_start, size_t i_end, wxTextCtrl *ctrl, wxWindow *parent, std::list<std::wstring> *val_out = NULL);
-
+
-protected:
+protected:
-    std::list<std::string> *m_val;  ///< Pointer to variable to receive control's parsed value
+    std::list<std::wstring> *m_val;  ///< Pointer to variable to receive control's parsed value
-};
+};
-
+
-
+
-class wxTLSCredentialsPanel : public wxEAPCredentialsPanelBase<eap::credentials_tls, wxTLSCredentialsPanelBase>
+class wxTLSCredentialsPanel : public wxEAPCredentialsPanelBase<eap::credentials_tls, wxTLSCredentialsPanelBase>
-{
+{
-public:
+public:
-    ///
+    ///
-    /// Constructs a configuration panel
+    /// Constructs a configuration panel
-    ///
+    ///
-    /// \param[in]    prov           Provider configuration data
+    /// \param[in]    prov           Provider configuration data
-    /// \param[in]    cfg            Configuration data
+    /// \param[in]    cfg            Configuration data
-    /// \param[inout] cred           Credentials data
+    /// \param[inout] cred           Credentials data
-    /// \param[in]    pszCredTarget  Target name of credentials in Windows Credential Manager. Can be further decorated to create final target name.
+    /// \param[in]    pszCredTarget  Target name of credentials in Windows Credential Manager. Can be further decorated to create final target name.
-    /// \param[in]    parent         Parent window
+    /// \param[in]    parent         Parent window
-    /// \param[in]    is_config      Is this panel used to pre-enter credentials? When \c true, the "Remember" checkbox is always selected and disabled.
+    /// \param[in]    is_config      Is this panel used to pre-enter credentials? When \c true, the "Remember" checkbox is always selected and disabled.
-    ///
+    ///
-    wxTLSCredentialsPanel(const eap::config_provider &prov, const eap::config_method_with_cred &cfg, eap::credentials_tls &cred, LPCTSTR pszCredTarget, wxWindow* parent, bool is_config = false);
+    wxTLSCredentialsPanel(const eap::config_provider &prov, const eap::config_method_with_cred &cfg, eap::credentials_tls &cred, LPCTSTR pszCredTarget, wxWindow* parent, bool is_config = false);
-
+
-protected:
+protected:
-    /// \cond internal
+    /// \cond internal
-    virtual bool TransferDataToWindow();
+    virtual bool TransferDataToWindow();
-    virtual bool TransferDataFromWindow();
+    virtual bool TransferDataFromWindow();
-    virtual void OnUpdateUI(wxUpdateUIEvent& event);
+    virtual void OnUpdateUI(wxUpdateUIEvent& event);
-    /// \endcond
+    /// \endcond
-
+
-protected:
+protected:
-    winstd::library m_shell32;      ///< shell32.dll resource library reference
+    winstd::library m_shell32;      ///< shell32.dll resource library reference
-    wxIcon m_icon;                  ///< Panel icon
+    wxIcon m_icon;                  ///< Panel icon
-};
+};
-
+
-
+
-class wxTLSServerTrustPanel : public wxEAPTLSServerTrustConfigPanelBase
+class wxTLSServerTrustPanel : public wxEAPTLSServerTrustConfigPanelBase
-{
+{
-public:
+public:
-    ///
+    ///
-    /// Constructs a configuration panel
+    /// Constructs a configuration panel
-    ///
+    ///
-    wxTLSServerTrustPanel(const eap::config_provider &prov, eap::config_method_tls &cfg, wxWindow* parent);
+    wxTLSServerTrustPanel(const eap::config_provider &prov, eap::config_method_tls &cfg, wxWindow* parent);
-
+
-protected:
+protected:
-    /// \cond internal
+    /// \cond internal
-    virtual bool TransferDataToWindow();
+    virtual bool TransferDataToWindow();
-    virtual bool TransferDataFromWindow();
+    virtual bool TransferDataFromWindow();
-    virtual void OnUpdateUI(wxUpdateUIEvent& event);
+    virtual void OnUpdateUI(wxUpdateUIEvent& event);
-    virtual void OnRootCADClick(wxCommandEvent& event);
+    virtual void OnRootCADClick(wxCommandEvent& event);
-    virtual void OnRootCAAddStore(wxCommandEvent& event);
+    virtual void OnRootCAAddStore(wxCommandEvent& event);
-    virtual void OnRootCAAddFile(wxCommandEvent& event);
+    virtual void OnRootCAAddFile(wxCommandEvent& event);
-    virtual void OnRootCARemove(wxCommandEvent& event);
+    virtual void OnRootCARemove(wxCommandEvent& event);
-    /// \endcond
+    /// \endcond
-
+
-    ///
+    ///
-    /// Adds a certificate to the list of trusted root CA list
+    /// Adds a certificate to the list of trusted root CA list
-    ///
+    ///
-    /// \param[in] cert  Certificate
+    /// \param[in] cert  Certificate
-    ///
+    ///
-    /// \returns
+    /// \returns
-    /// - \c true  if certificate was added;
+    /// - \c true  if certificate was added;
-    /// - \c false if duplicate found or an error occured.
+    /// - \c false if duplicate found or an error occured.
-    ///
+    ///
-    bool AddRootCA(PCCERT_CONTEXT cert);
+    bool AddRootCA(PCCERT_CONTEXT cert);
-
+
-protected:
+protected:
-    const eap::config_provider &m_prov;         ///< EAP provider
+    const eap::config_provider &m_prov;         ///< EAP provider
-    eap::config_method_tls &m_cfg;              ///< TLS configuration
+    eap::config_method_tls &m_cfg;              ///< TLS configuration
-    winstd::library m_certmgr;                  ///< certmgr.dll resource library reference
+    winstd::library m_certmgr;                  ///< certmgr.dll resource library reference
-    wxIcon m_icon;                              ///< Panel icon
+    wxIcon m_icon;                              ///< Panel icon
-    std::list<std::string> m_server_names_val;  ///< Acceptable authenticating server names
+    std::list<std::wstring> m_server_names_val; ///< Acceptable authenticating server names
-};
+};
-
+
-
+
-class wxTLSConfigPanel : public wxPanel
+class wxTLSConfigPanel : public wxPanel
-{
+{
-public:
+public:
-    ///
+    ///
-    /// Constructs a configuration panel
+    /// Constructs a configuration panel
-    ///
+    ///
-    wxTLSConfigPanel(const eap::config_provider &prov, eap::config_method_tls &cfg, LPCTSTR pszCredTarget, wxWindow* parent);
+    wxTLSConfigPanel(const eap::config_provider &prov, eap::config_method_tls &cfg, LPCTSTR pszCredTarget, wxWindow* parent);
-
+
-    ///
+    ///
-    /// Destructs the configuration panel
+    /// Destructs the configuration panel
-    ///
+    ///
-    virtual ~wxTLSConfigPanel();
+    virtual ~wxTLSConfigPanel();
-
+
-protected:
+protected:
-    /// \cond internal
+    /// \cond internal
-    virtual void OnInitDialog(wxInitDialogEvent& event);
+    virtual void OnInitDialog(wxInitDialogEvent& event);
-    virtual bool TransferDataFromWindow();
+    virtual bool TransferDataFromWindow();
-    /// \endcond
+    /// \endcond
-
+
-protected:
+protected:
-    const eap::config_provider &m_prov;          ///< EAP provider
+    const eap::config_provider &m_prov;          ///< EAP provider
-    eap::config_method_tls &m_cfg;               ///< TLS configuration
+    eap::config_method_tls &m_cfg;               ///< TLS configuration
-    wxTLSServerTrustPanel *m_server_trust;       ///< Server trust configuration panel
+    wxTLSServerTrustPanel *m_server_trust;       ///< Server trust configuration panel
-    wxTLSCredentialsConfigPanel *m_credentials;  ///< Credentials configuration panel
+    wxTLSCredentialsConfigPanel *m_credentials;  ///< Credentials configuration panel
-};
+};

lib/TLS_UI/res/wxTLS_UI.cpp

@@ -1,182 +1,182 @@
-///////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
-// C++ code generated with wxFormBuilder (version Jun 17 2015)
+// C++ code generated with wxFormBuilder (version Jun 17 2015)
-// http://www.wxformbuilder.org/
+// http://www.wxformbuilder.org/
-//
+//
-// PLEASE DO "NOT" EDIT THIS FILE!
+// PLEASE DO "NOT" EDIT THIS FILE!
-///////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
-
+
-#include <StdAfx.h>
+#include <StdAfx.h>
-
+
-#include "wxTLS_UI.h"
+#include "wxTLS_UI.h"
-
+
-///////////////////////////////////////////////////////////////////////////
+///////////////////////////////////////////////////////////////////////////
-
+
-wxEAPTLSServerTrustConfigPanelBase::wxEAPTLSServerTrustConfigPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
+wxEAPTLSServerTrustConfigPanelBase::wxEAPTLSServerTrustConfigPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
-{
+{
-	wxStaticBoxSizer* sb_server_trust;
+	wxStaticBoxSizer* sb_server_trust;
-	sb_server_trust = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("Server Trust") ), wxVERTICAL );
+	sb_server_trust = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("Server Trust") ), wxVERTICAL );
-	
+	
-	wxBoxSizer* sb_server_trust_horiz;
+	wxBoxSizer* sb_server_trust_horiz;
-	sb_server_trust_horiz = new wxBoxSizer( wxHORIZONTAL );
+	sb_server_trust_horiz = new wxBoxSizer( wxHORIZONTAL );
-	
+	
-	m_server_trust_icon = new wxStaticBitmap( sb_server_trust->GetStaticBox(), wxID_ANY, wxNullBitmap, wxDefaultPosition, wxDefaultSize, 0 );
+	m_server_trust_icon = new wxStaticBitmap( sb_server_trust->GetStaticBox(), wxID_ANY, wxNullBitmap, wxDefaultPosition, wxDefaultSize, 0 );
-	sb_server_trust_horiz->Add( m_server_trust_icon, 0, wxALL, 5 );
+	sb_server_trust_horiz->Add( m_server_trust_icon, 0, wxALL, 5 );
-	
+	
-	wxBoxSizer* sb_server_trust_vert;
+	wxBoxSizer* sb_server_trust_vert;
-	sb_server_trust_vert = new wxBoxSizer( wxVERTICAL );
+	sb_server_trust_vert = new wxBoxSizer( wxVERTICAL );
-	
+	
-	m_server_trust_label = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("Describe the servers you trust to prevent credential interception in case of man-in-the-middle attacks."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_server_trust_label = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("Describe the servers you trust to prevent credential interception in case of man-in-the-middle attacks."), wxDefaultPosition, wxDefaultSize, 0 );
-	m_server_trust_label->Wrap( 446 );
+	m_server_trust_label->Wrap( 446 );
-	sb_server_trust_vert->Add( m_server_trust_label, 0, wxALL|wxEXPAND, 5 );
+	sb_server_trust_vert->Add( m_server_trust_label, 0, wxALL|wxEXPAND, 5 );
-	
+	
-	wxBoxSizer* sb_root_ca;
+	wxBoxSizer* sb_root_ca;
-	sb_root_ca = new wxBoxSizer( wxVERTICAL );
+	sb_root_ca = new wxBoxSizer( wxVERTICAL );
-	
+	
-	m_root_ca_lbl = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("Acceptable Certificate Authorities:"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_root_ca_lbl = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("Acceptable Certificate Authorities:"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_root_ca_lbl->Wrap( -1 );
+	m_root_ca_lbl->Wrap( -1 );
-	sb_root_ca->Add( m_root_ca_lbl, 0, wxEXPAND|wxBOTTOM, 5 );
+	sb_root_ca->Add( m_root_ca_lbl, 0, wxEXPAND|wxBOTTOM, 5 );
-	
+	
-	m_root_ca = new wxListBox( sb_server_trust->GetStaticBox(), wxID_ANY, wxDefaultPosition, wxDefaultSize, 0, NULL, wxLB_SORT ); 
+	m_root_ca = new wxListBox( sb_server_trust->GetStaticBox(), wxID_ANY, wxDefaultPosition, wxDefaultSize, 0, NULL, wxLB_SORT ); 
-	m_root_ca->SetToolTip( _("List of certificate authorities server's certificate must be issued by") );
+	m_root_ca->SetToolTip( _("List of certificate authorities server's certificate must be issued by") );
-	
+	
-	sb_root_ca->Add( m_root_ca, 1, wxEXPAND|wxBOTTOM, 5 );
+	sb_root_ca->Add( m_root_ca, 1, wxEXPAND|wxBOTTOM, 5 );
-	
+	
-	wxBoxSizer* sb_root_ca_btn;
+	wxBoxSizer* sb_root_ca_btn;
-	sb_root_ca_btn = new wxBoxSizer( wxHORIZONTAL );
+	sb_root_ca_btn = new wxBoxSizer( wxHORIZONTAL );
-	
+	
-	m_root_ca_add_store = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("Add CA from Store..."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_root_ca_add_store = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("Add CA from Store..."), wxDefaultPosition, wxDefaultSize, 0 );
-	m_root_ca_add_store->SetToolTip( _("Adds a new certificate authority from the certificate store to the list") );
+	m_root_ca_add_store->SetToolTip( _("Adds a new certificate authority from the certificate store to the list") );
-	
+	
-	sb_root_ca_btn->Add( m_root_ca_add_store, 0, wxRIGHT, 5 );
+	sb_root_ca_btn->Add( m_root_ca_add_store, 0, wxRIGHT, 5 );
-	
+	
-	m_root_ca_add_file = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("Add CA from File..."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_root_ca_add_file = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("Add CA from File..."), wxDefaultPosition, wxDefaultSize, 0 );
-	m_root_ca_add_file->SetToolTip( _("Adds a new certificate authority from the file to the list") );
+	m_root_ca_add_file->SetToolTip( _("Adds a new certificate authority from the file to the list") );
-	
+	
-	sb_root_ca_btn->Add( m_root_ca_add_file, 0, wxRIGHT|wxLEFT, 5 );
+	sb_root_ca_btn->Add( m_root_ca_add_file, 0, wxRIGHT|wxLEFT, 5 );
-	
+	
-	m_root_ca_remove = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("&Remove CA"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_root_ca_remove = new wxButton( sb_server_trust->GetStaticBox(), wxID_ANY, _("&Remove CA"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_root_ca_remove->Enable( false );
+	m_root_ca_remove->Enable( false );
-	m_root_ca_remove->SetToolTip( _("Removes selected certificate authorities from the list") );
+	m_root_ca_remove->SetToolTip( _("Removes selected certificate authorities from the list") );
-	
+	
-	sb_root_ca_btn->Add( m_root_ca_remove, 0, wxLEFT, 5 );
+	sb_root_ca_btn->Add( m_root_ca_remove, 0, wxLEFT, 5 );
-	
+	
-	
+	
-	sb_root_ca->Add( sb_root_ca_btn, 0, wxALIGN_RIGHT, 5 );
+	sb_root_ca->Add( sb_root_ca_btn, 0, wxALIGN_RIGHT, 5 );
-	
+	
-	
+	
-	sb_server_trust_vert->Add( sb_root_ca, 1, wxEXPAND|wxALL, 5 );
+	sb_server_trust_vert->Add( sb_root_ca, 1, wxEXPAND|wxALL, 5 );
-	
+	
-	wxBoxSizer* sb_server_names;
+	wxBoxSizer* sb_server_names;
-	sb_server_names = new wxBoxSizer( wxVERTICAL );
+	sb_server_names = new wxBoxSizer( wxVERTICAL );
-	
+	
-	m_server_names_label = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("Acceptable server &names:"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_server_names_label = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("Acceptable server &names:"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_server_names_label->Wrap( -1 );
+	m_server_names_label->Wrap( -1 );
-	sb_server_names->Add( m_server_names_label, 0, wxBOTTOM, 5 );
+	sb_server_names->Add( m_server_names_label, 0, wxBOTTOM, 5 );
-	
+	
-	m_server_names = new wxTextCtrl( sb_server_trust->GetStaticBox(), wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, 0 );
+	m_server_names = new wxTextCtrl( sb_server_trust->GetStaticBox(), wxID_ANY, wxEmptyString, wxDefaultPosition, wxDefaultSize, 0 );
-	m_server_names->SetToolTip( _("A semicolon delimited list of acceptable server FQDN names; blank to skip name check; \"*\" wildchar allowed") );
+	m_server_names->SetToolTip( _("A semicolon delimited list of acceptable server FQDN names; blank to skip name check; Unicode characters allowed") );
-	
+	
-	sb_server_names->Add( m_server_names, 0, wxEXPAND|wxBOTTOM, 5 );
+	sb_server_names->Add( m_server_names, 0, wxEXPAND|wxBOTTOM, 5 );
-	
+	
-	m_server_names_note = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("(Example: foo.bar.com;*.domain.org)"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_server_names_note = new wxStaticText( sb_server_trust->GetStaticBox(), wxID_ANY, _("(Example: foo.bar.com;server2.bar.com)"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_server_names_note->Wrap( -1 );
+	m_server_names_note->Wrap( -1 );
-	sb_server_names->Add( m_server_names_note, 0, wxALIGN_RIGHT, 5 );
+	sb_server_names->Add( m_server_names_note, 0, wxALIGN_RIGHT, 5 );
-	
+	
-	
+	
-	sb_server_trust_vert->Add( sb_server_names, 0, wxEXPAND|wxALL, 5 );
+	sb_server_trust_vert->Add( sb_server_names, 0, wxEXPAND|wxALL, 5 );
-	
+	
-	
+	
-	sb_server_trust_horiz->Add( sb_server_trust_vert, 1, wxEXPAND, 5 );
+	sb_server_trust_horiz->Add( sb_server_trust_vert, 1, wxEXPAND, 5 );
-	
+	
-	
+	
-	sb_server_trust->Add( sb_server_trust_horiz, 1, wxEXPAND, 5 );
+	sb_server_trust->Add( sb_server_trust_horiz, 1, wxEXPAND, 5 );
-	
+	
-	
+	
-	this->SetSizer( sb_server_trust );
+	this->SetSizer( sb_server_trust );
-	this->Layout();
+	this->Layout();
-	
+	
-	// Connect Events
+	// Connect Events
-	this->Connect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnUpdateUI ) );
+	this->Connect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnUpdateUI ) );
-	m_root_ca->Connect( wxEVT_COMMAND_LISTBOX_DOUBLECLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCADClick ), NULL, this );
+	m_root_ca->Connect( wxEVT_COMMAND_LISTBOX_DOUBLECLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCADClick ), NULL, this );
-	m_root_ca_add_store->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddStore ), NULL, this );
+	m_root_ca_add_store->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddStore ), NULL, this );
-	m_root_ca_add_file->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddFile ), NULL, this );
+	m_root_ca_add_file->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddFile ), NULL, this );
-	m_root_ca_remove->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCARemove ), NULL, this );
+	m_root_ca_remove->Connect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCARemove ), NULL, this );
-}
+}
-
+
-wxEAPTLSServerTrustConfigPanelBase::~wxEAPTLSServerTrustConfigPanelBase()
+wxEAPTLSServerTrustConfigPanelBase::~wxEAPTLSServerTrustConfigPanelBase()
-{
+{
-	// Disconnect Events
+	// Disconnect Events
-	this->Disconnect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnUpdateUI ) );
+	this->Disconnect( wxEVT_UPDATE_UI, wxUpdateUIEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnUpdateUI ) );
-	m_root_ca->Disconnect( wxEVT_COMMAND_LISTBOX_DOUBLECLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCADClick ), NULL, this );
+	m_root_ca->Disconnect( wxEVT_COMMAND_LISTBOX_DOUBLECLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCADClick ), NULL, this );
-	m_root_ca_add_store->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddStore ), NULL, this );
+	m_root_ca_add_store->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddStore ), NULL, this );
-	m_root_ca_add_file->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddFile ), NULL, this );
+	m_root_ca_add_file->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCAAddFile ), NULL, this );
-	m_root_ca_remove->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCARemove ), NULL, this );
+	m_root_ca_remove->Disconnect( wxEVT_COMMAND_BUTTON_CLICKED, wxCommandEventHandler( wxEAPTLSServerTrustConfigPanelBase::OnRootCARemove ), NULL, this );
-	
+	
-}
+}
-
+
-wxTLSCredentialsPanelBase::wxTLSCredentialsPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
+wxTLSCredentialsPanelBase::wxTLSCredentialsPanelBase( wxWindow* parent, wxWindowID id, const wxPoint& pos, const wxSize& size, long style ) : wxPanel( parent, id, pos, size, style )
-{
+{
-	wxStaticBoxSizer* sb_credentials;
+	wxStaticBoxSizer* sb_credentials;
-	sb_credentials = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("TLS Client Certificate") ), wxVERTICAL );
+	sb_credentials = new wxStaticBoxSizer( new wxStaticBox( this, wxID_ANY, _("TLS Client Certificate") ), wxVERTICAL );
-	
+	
-	wxBoxSizer* sb_credentials_horiz;
+	wxBoxSizer* sb_credentials_horiz;
-	sb_credentials_horiz = new wxBoxSizer( wxHORIZONTAL );
+	sb_credentials_horiz = new wxBoxSizer( wxHORIZONTAL );
-	
+	
-	m_credentials_icon = new wxStaticBitmap( sb_credentials->GetStaticBox(), wxID_ANY, wxNullBitmap, wxDefaultPosition, wxDefaultSize, 0 );
+	m_credentials_icon = new wxStaticBitmap( sb_credentials->GetStaticBox(), wxID_ANY, wxNullBitmap, wxDefaultPosition, wxDefaultSize, 0 );
-	sb_credentials_horiz->Add( m_credentials_icon, 0, wxALL, 5 );
+	sb_credentials_horiz->Add( m_credentials_icon, 0, wxALL, 5 );
-	
+	
-	wxBoxSizer* sb_credentials_vert;
+	wxBoxSizer* sb_credentials_vert;
-	sb_credentials_vert = new wxBoxSizer( wxVERTICAL );
+	sb_credentials_vert = new wxBoxSizer( wxVERTICAL );
-	
+	
-	m_credentials_label = new wxStaticText( sb_credentials->GetStaticBox(), wxID_ANY, _("Please select your client certificate to use for authentication."), wxDefaultPosition, wxDefaultSize, 0 );
+	m_credentials_label = new wxStaticText( sb_credentials->GetStaticBox(), wxID_ANY, _("Please select your client certificate to use for authentication."), wxDefaultPosition, wxDefaultSize, 0 );
-	m_credentials_label->Wrap( 446 );
+	m_credentials_label->Wrap( 446 );
-	sb_credentials_vert->Add( m_credentials_label, 0, wxALL|wxEXPAND, 5 );
+	sb_credentials_vert->Add( m_credentials_label, 0, wxALL|wxEXPAND, 5 );
-	
+	
-	wxBoxSizer* sb_cert_radio;
+	wxBoxSizer* sb_cert_radio;
-	sb_cert_radio = new wxBoxSizer( wxVERTICAL );
+	sb_cert_radio = new wxBoxSizer( wxVERTICAL );
-	
+	
-	m_cert_none = new wxRadioButton( sb_credentials->GetStaticBox(), wxID_ANY, _("Co&nnect without providing a client certificate"), wxDefaultPosition, wxDefaultSize, wxRB_GROUP );
+	m_cert_none = new wxRadioButton( sb_credentials->GetStaticBox(), wxID_ANY, _("Co&nnect without providing a client certificate"), wxDefaultPosition, wxDefaultSize, wxRB_GROUP );
-	m_cert_none->SetToolTip( _("Select if your server does not require you to provide a client certificate") );
+	m_cert_none->SetToolTip( _("Select if your server does not require you to provide a client certificate") );
-	
+	
-	sb_cert_radio->Add( m_cert_none, 1, wxEXPAND, 5 );
+	sb_cert_radio->Add( m_cert_none, 1, wxEXPAND, 5 );
-	
+	
-	wxBoxSizer* sb_cert_select;
+	wxBoxSizer* sb_cert_select;
-	sb_cert_select = new wxBoxSizer( wxHORIZONTAL );
+	sb_cert_select = new wxBoxSizer( wxHORIZONTAL );
-	
+	
-	m_cert_select = new wxRadioButton( sb_credentials->GetStaticBox(), wxID_ANY, _("Use the following &certificate:"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_cert_select = new wxRadioButton( sb_credentials->GetStaticBox(), wxID_ANY, _("Use the following &certificate:"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_cert_select->SetToolTip( _("Select if you need to provide a client certificate when connecting") );
+	m_cert_select->SetToolTip( _("Select if you need to provide a client certificate when connecting") );
-	
+	
-	sb_cert_select->Add( m_cert_select, 0, wxEXPAND, 5 );
+	sb_cert_select->Add( m_cert_select, 0, wxEXPAND, 5 );
-	
+	
-	wxArrayString m_cert_select_valChoices;
+	wxArrayString m_cert_select_valChoices;
-	m_cert_select_val = new wxChoice( sb_credentials->GetStaticBox(), wxID_ANY, wxDefaultPosition, wxDefaultSize, m_cert_select_valChoices, wxCB_SORT );
+	m_cert_select_val = new wxChoice( sb_credentials->GetStaticBox(), wxID_ANY, wxDefaultPosition, wxDefaultSize, m_cert_select_valChoices, wxCB_SORT );
-	m_cert_select_val->SetSelection( 0 );
+	m_cert_select_val->SetSelection( 0 );
-	m_cert_select_val->SetToolTip( _("Client certificate to use for authentication") );
+	m_cert_select_val->SetToolTip( _("Client certificate to use for authentication") );
-	
+	
-	sb_cert_select->Add( m_cert_select_val, 1, wxEXPAND, 5 );
+	sb_cert_select->Add( m_cert_select_val, 1, wxEXPAND, 5 );
-	
+	
-	
+	
-	sb_cert_radio->Add( sb_cert_select, 1, wxEXPAND, 5 );
+	sb_cert_radio->Add( sb_cert_select, 1, wxEXPAND, 5 );
-	
+	
-	
+	
-	sb_credentials_vert->Add( sb_cert_radio, 0, wxEXPAND|wxALL, 5 );
+	sb_credentials_vert->Add( sb_cert_radio, 0, wxEXPAND|wxALL, 5 );
-	
+	
-	m_remember = new wxCheckBox( sb_credentials->GetStaticBox(), wxID_ANY, _("&Remember"), wxDefaultPosition, wxDefaultSize, 0 );
+	m_remember = new wxCheckBox( sb_credentials->GetStaticBox(), wxID_ANY, _("&Remember"), wxDefaultPosition, wxDefaultSize, 0 );
-	m_remember->SetHelpText( _("Check if you would like to save certificate selection") );
+	m_remember->SetHelpText( _("Check if you would like to save certificate selection") );
-	
+	
-	sb_credentials_vert->Add( m_remember, 0, wxALL|wxEXPAND, 5 );
+	sb_credentials_vert->Add( m_remember, 0, wxALL|wxEXPAND, 5 );
-	
+	
-	
+	
-	sb_credentials_horiz->Add( sb_credentials_vert, 1, wxEXPAND, 5 );
+	sb_credentials_horiz->Add( sb_credentials_vert, 1, wxEXPAND, 5 );
-	
+	
-	
+	
-	sb_credentials->Add( sb_credentials_horiz, 0, wxEXPAND, 5 );
+	sb_credentials->Add( sb_credentials_horiz, 0, wxEXPAND, 5 );
-	
+	
-	
+	
-	this->SetSizer( sb_credentials );
+	this->SetSizer( sb_credentials );
-	this->Layout();
+	this->Layout();
-}
+}
-
+
-wxTLSCredentialsPanelBase::~wxTLSCredentialsPanelBase()
+wxTLSCredentialsPanelBase::~wxTLSCredentialsPanelBase()
-{
+{
-}
+}

lib/TTLS_UI/include/TTLS_UI.h

@@ -45,7 +45,6 @@ class wxTTLSCredentialsPanel;
 
 #include <wx/choicebk.h>
 #include <wx/icon.h>
-#include <wx/scrolwin.h>
 #include <wx/stattext.h>
 
 #include <Windows.h>
@@ -74,32 +73,28 @@ protected:
 };
 
 
-class wxTTLSConfigWindow : public wxScrolledWindow
+class wxTTLSConfigWindow : public wxEAPConfigWindow
 {
 public:
     ///
     /// Constructs a configuration panel
     ///
+    /// \param[in]    prov           Provider configuration data
     /// \param[inout] cfg            Configuration data
     /// \param[in]    pszCredTarget  Target name of credentials in Windows Credential Manager. Can be further decorated to create final target name.
     /// \param[in]    parent         Parent window
     ///
     wxTTLSConfigWindow(const eap::config_provider &prov, eap::config_method &cfg, LPCTSTR pszCredTarget, wxWindow* parent);
 
-    ///
-    /// Destructs the configuration panel
-    ///
-    virtual ~wxTTLSConfigWindow();
-
 protected:
     /// \cond internal
     virtual bool TransferDataToWindow();
     virtual bool TransferDataFromWindow();
     virtual void OnInitDialog(wxInitDialogEvent& event);
+    virtual void OnUpdateUI(wxUpdateUIEvent& event);
     /// \endcond
 
 protected:
-    const eap::config_provider &m_prov;     ///< EAP provider
     eap::config_method_ttls &m_cfg;         ///< TTLS configuration
     wxStaticText *m_outer_title;            ///< Outer authentication title
     wxTTLSConfigPanel *m_outer_identity;    ///< Outer identity configuration panel
@@ -108,7 +103,7 @@ protected:
     wxChoicebook *m_inner_type;             ///< Inner authentication type
 
     // Temporary inner method configurations to hold data until applied
-    eap::config_method_pap m_cfg_pap;            ///< PAP configuration
+    eap::config_method_pap m_cfg_pap;       ///< PAP configuration
 };
 
 

lib/TTLS_UI/src/Module.cpp

@@ -83,8 +83,26 @@ void eap::peer_ttls_ui::invoke_config_ui(
 {
     // Unpack configuration.
     config_provider_list cfg(*this);
-    if (dwConnectionDataInSize)
+    if (dwConnectionDataInSize) {
+        // Load existing configuration.
         unpack(cfg, pConnectionDataIn, dwConnectionDataInSize);
+    } else {
+        // This is a blank network profile. Create default configuraton.
+
+        // Start with PAP inner configuration.
+        unique_ptr<config_method_ttls> cfg_method(new config_method_ttls(*this));
+        cfg_method->m_inner.reset(new config_method_pap(*this));
+        cfg_method->m_anonymous_identity = L"@";
+        cfg_method->m_use_preshared = true;
+        cfg_method->m_preshared.reset(new credentials_tls(*this));
+
+        // Start with one method.
+        config_provider cfg_provider(*this);
+        cfg_provider.m_methods.push_back(std::move(cfg_method));
+
+        // Start with one provider.
+        cfg.m_providers.push_back(std::move(cfg_provider));
+    }
 
     // Initialize application.
     new wxApp();
@@ -193,7 +211,7 @@ void eap::peer_ttls_ui::invoke_identity_ui(
         // Create credentials dialog.
         wxEAPCredentialsDialog dlg(cfg_prov, &parent);
         wxTTLSCredentialsPanel *panel = new wxTTLSCredentialsPanel(cfg_prov, *cfg_method, cred_out, cfg_prov.m_id.c_str(), &dlg);
-        dlg.AddContents((wxPanel**)&panel, 1);
+        dlg.AddContent(panel);
 
         // Set "Remember" checkboxes according to credential source,
         panel->m_outer_cred->SetRememberValue(cred_source.first == eap::credentials::source_storage);

lib/TTLS_UI/src/TTLS_UI.cpp

@@ -38,14 +38,6 @@ wxTTLSConfigPanel::wxTTLSConfigPanel(const eap::config_provider &prov, eap::conf
 
 bool wxTTLSConfigPanel::TransferDataToWindow()
 {
-    if (m_prov.m_read_only) {
-        // This is provider-locked configuration. Disable controls.
-        m_outer_identity_same      ->Enable(false);
-        m_outer_identity_empty     ->Enable(false);
-        m_outer_identity_custom    ->Enable(false);
-        m_outer_identity_custom_val->Enable(false);
-    }
-
     // Populate identity controls.
     if (m_cfg.m_anonymous_identity.empty()) {
         m_outer_identity_same->SetValue(true);
@@ -82,8 +74,17 @@ void wxTTLSConfigPanel::OnUpdateUI(wxUpdateUIEvent& event)
 {
     UNREFERENCED_PARAMETER(event);
 
-    if (!m_prov.m_read_only) {
+    if (m_prov.m_read_only) {
+        // This is provider-locked configuration. Disable controls.
+        m_outer_identity_same      ->Enable(false);
+        m_outer_identity_empty     ->Enable(false);
+        m_outer_identity_custom    ->Enable(false);
+        m_outer_identity_custom_val->Enable(false);
+    } else {
         // This is not a provider-locked configuration. Selectively enable/disable controls.
+        m_outer_identity_same      ->Enable(true);
+        m_outer_identity_empty     ->Enable(true);
+        m_outer_identity_custom    ->Enable(true);
         m_outer_identity_custom_val->Enable(m_outer_identity_custom->GetValue());
     }
 }
@@ -94,10 +95,9 @@ void wxTTLSConfigPanel::OnUpdateUI(wxUpdateUIEvent& event)
 //////////////////////////////////////////////////////////////////////
 
 wxTTLSConfigWindow::wxTTLSConfigWindow(const eap::config_provider &prov, eap::config_method &cfg, LPCTSTR pszCredTarget, wxWindow* parent) :
-    m_prov(prov),
     m_cfg((eap::config_method_ttls&)cfg),
     m_cfg_pap(cfg.m_module),
-    wxScrolledWindow(parent, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxVSCROLL)
+    wxEAPConfigWindow(prov, cfg, parent)
 {
     wxBoxSizer* sb_content;
     sb_content = new wxBoxSizer( wxVERTICAL );
@@ -112,7 +112,8 @@ wxTTLSConfigWindow::wxTTLSConfigWindow(const eap::config_provider &prov, eap::co
 
     m_inner_type = new wxChoicebook(this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxCHB_DEFAULT);
     m_inner_type->SetToolTip( _("Select inner authentication method from the list") );
-    m_inner_type->AddPage(new wxPAPConfigPanel(m_prov, m_cfg_pap, pszCredTarget, m_inner_type), _("PAP"));
+    wxPAPConfigPanel *panel_pap = new wxPAPConfigPanel(m_prov, m_cfg_pap, pszCredTarget, m_inner_type);
+    m_inner_type->AddPage(panel_pap, _("PAP"));
     sb_content->Add(m_inner_type, 0, wxALL|wxEXPAND, 5);
 
     sb_content->Add(20, 20, 1, wxALL|wxEXPAND, 5);
@@ -135,32 +136,17 @@ wxTTLSConfigWindow::wxTTLSConfigWindow(const eap::config_provider &prov, eap::co
         size.y  = 500;
     }
     this->SetMinSize(size);
-    this->SetScrollRate(5, 5);
 
     this->SetSizer(sb_content);
     this->Layout();
 
-    m_inner_type->SetFocusFromKbd();
+    // m_inner_type->SetFocusFromKbd(); // This control steals mouse-wheel scrolling for itself
-
+    panel_pap->SetFocusFromKbd();
-    // Connect Events
-    this->Connect(wxEVT_INIT_DIALOG, wxInitDialogEventHandler(wxTTLSConfigWindow::OnInitDialog));
-}
-
-
-wxTTLSConfigWindow::~wxTTLSConfigWindow()
-{
-    // Disconnect Events
-    this->Disconnect(wxEVT_INIT_DIALOG, wxInitDialogEventHandler(wxTTLSConfigWindow::OnInitDialog));
 }
 
 
 bool wxTTLSConfigWindow::TransferDataToWindow()
 {
-    if (m_prov.m_read_only) {
-        // This is provider-locked configuration. Disable controls.
-        m_inner_type->GetChoiceCtrl()->Enable(false);
-    }
-
     eap::config_method_pap *cfg_pap = dynamic_cast<eap::config_method_pap*>(m_cfg.m_inner.get());
     if (cfg_pap) {
         m_cfg_pap = *cfg_pap;
@@ -196,8 +182,7 @@ bool wxTTLSConfigWindow::TransferDataFromWindow()
 
 void wxTTLSConfigWindow::OnInitDialog(wxInitDialogEvent& event)
 {
-    // Call TransferDataToWindow() manually, as wxScrolledWindow somehow skips that.
+    wxEAPConfigWindow::OnInitDialog(event);
-    TransferDataToWindow();
 
     // Forward the event to child panels.
     m_outer_identity->GetEventHandler()->ProcessEvent(event);
@@ -207,6 +192,14 @@ void wxTTLSConfigWindow::OnInitDialog(wxInitDialogEvent& event)
 }
 
 
+void wxTTLSConfigWindow::OnUpdateUI(wxUpdateUIEvent& event)
+{
+    wxEAPConfigWindow::OnUpdateUI(event);
+
+    m_inner_type->GetChoiceCtrl()->Enable(!m_prov.m_read_only);
+}
+
+
 //////////////////////////////////////////////////////////////////////
 // wxTTLSCredentialsPanel
 //////////////////////////////////////////////////////////////////////