Implemented "Clear Trust Anchor" button in Edit Identity Dialog.
[moonshot-ui.git] / src / moonshot-identity-manager-app.vala
index 028773f..4de64be 100644 (file)
@@ -43,14 +43,19 @@ interface IIdentityManager : GLib.Object {
 }
 #endif
 
+
 public class IdentityManagerApp {
+    public static MoonshotLogger logger = get_logger("IdentityManagerApp");
+
     public IdentityManagerModel model;
     public IdCard default_id_card;
     public bool explicitly_launched;
     public IdentityManagerView view;
     private MoonshotServer ipc_server;
+    private bool name_is_owned;
+    private bool show_requested;
 
-    #if OS_MACOS
+#if OS_MACOS
     public OSXApplication osxApp;
   
     // the signal handler function.
@@ -62,22 +67,33 @@ public class IdentityManagerApp {
         int added_cards = id_manager_app_instance.ipc_server.install_from_file(file_name);
         return true;
     }
-    #endif
+#endif
 
-    private const int WINDOW_WIDTH = 400;
-    private const int WINDOW_HEIGHT = 500;
+    /** If we're successfully registered with DBus, then show the UI. Otherwise, wait until we're registered. */
     public void show() {
-        if (view != null) view.make_visible();
+        if (name_is_owned) {
+            if (view != null) {
+                view.make_visible();
+            }
+        }
+        else {
+            show_requested = true;
+        }
     }
+    
+#if LOG4VALA
+    // Call this from main() to ensure that the logger is initialized
+    internal IdentityManagerApp.dummy() {}
+#endif
 
     public IdentityManagerApp(bool headless, bool use_flat_file_store) {
         use_flat_file_store |= UserForcesFlatFileStore();
 
-        #if GNOME_KEYRING
-            bool keyring_available = (!use_flat_file_store) && GnomeKeyring.is_available();
-        #else
-            bool keyring_available = false;
-        #endif
+#if GNOME_KEYRING
+        bool keyring_available = (!use_flat_file_store) && GnomeKeyring.is_available();
+#else
+        bool keyring_available = false;
+#endif
 
         IIdentityCardStore.StoreType store_type;
         if (headless || use_flat_file_store || !keyring_available)
@@ -99,22 +115,30 @@ public class IdentityManagerApp {
 
         init_ipc_server();
 
-        #if OS_MACOS
+#if OS_MACOS
         osxApp = OSXApplication.get_instance();
         // The 'correct' way of connecting won't work in Mac OS with Vala 0.12; e.g.
         //     osxApp.ns_application_open_file.connect(install_from_file);
         // so we have to use this old way
         Signal.connect(osxApp, "NSApplicationOpenFile", (GLib.Callback)(on_osx_open_files), this);
-        #endif
+#endif
     }
 
     public bool add_identity(IdCard id, bool force_flat_file_store) {
-        if (view != null) return view.add_identity(id, force_flat_file_store);
+        if (view != null) 
+        {
+            logger.trace("add_identity: calling view.add_identity");
+            return view.add_identity(id, force_flat_file_store);
+        }
+
+        logger.trace("add_identity: calling model.add_card");
         model.add_card(id, force_flat_file_store);
         return true;
     }
 
     public void select_identity(IdentityRequest request) {
+        logger.trace("select_identity: request.nai=%s".printf(request.nai ?? "[null]"));
+
         IdCard identity = null;
 
         if (request.select_default)
@@ -133,6 +157,7 @@ public class IdentityManagerApp {
                 /* If NAI matches, use this id card */
                 if (has_nai && request.nai == id.nai)
                 {
+                    logger.trace("select_identity: request has nai; returning " + id.display_name);
                     identity = id;
                     break;
                 }
@@ -140,13 +165,9 @@ public class IdentityManagerApp {
                 /* If any service matches we add id card to the candidate list */
                 if (has_srv)
                 {
-                    foreach (string srv in id.services)
-                    {
-                        if (request.service == srv)
-                        {
-                            request.candidates.append(id);
-                            continue;
-                        }
+                    if (id.services.contains(request.service)) {
+                        logger.trace(@"select_identity: request has service '$(request.service); matched on '$(id.display_name)'");
+                        request.candidates.append(id);
                     }
                 }
             }
@@ -154,45 +175,17 @@ public class IdentityManagerApp {
             /* If more than one candidate we dissasociate service from all ids */
             if ((identity == null) && has_srv && request.candidates.length() > 1)
             {
+                logger.trace(@"select_identity: multiple candidates; removing service '$(request.service) from all.");
                 foreach (IdCard id in request.candidates)
                 {
-                    int i = 0;
-                    SList<string> services_list = null;
-                    bool has_service = false;
-
-                    foreach (string srv in id.services)
-                    {
-                        if (srv == request.service)
-                        {
-                            has_service = true;
-                            continue;
-                        }
-                        services_list.append(srv);
-                    }
-                    
-                    if (!has_service)
-                        continue;
-
-                    if (services_list.length() == 0)
-                    {
-                        id.services = {};
-                        continue;
-                    }
-
-                    string[] services = new string[services_list.length()];
-                    foreach (string srv in services_list)
-                    {
-                        services[i] = srv;
-                        i++;
-                    }
-
-                    id.services = services;
+                    id.services.remove(request.service);
                 }
             }
 
             /* If there are no candidates we use the service matching rules */
             if ((identity == null) && (request.candidates.length() == 0))
             {
+                logger.trace("select_identity: No candidates; using service matching rules.");
                 foreach (IdCard id in model.get_card_list())
                 {
                     foreach (Rule rule in id.rules)
@@ -200,6 +193,7 @@ public class IdentityManagerApp {
                         if (!match_service_pattern(request.service, rule.pattern))
                             continue;
 
+                        logger.trace(@"select_identity: ID $(id.display_name) matched on service matching rules.");
                         request.candidates.append(id);
 
                         if (rule.always_confirm == "true")
@@ -209,6 +203,7 @@ public class IdentityManagerApp {
             }
             
             if ((identity == null) && has_nai) {
+                logger.trace("select_identity: Creating temp identity");
                 // create a temp identity
                 string[] components = request.nai.split("@", 2);
                 identity = new IdCard();
@@ -221,6 +216,7 @@ public class IdentityManagerApp {
             }
             if (identity == null) {
                 if (request.candidates.length() != 1) {
+                    logger.trace("select_identity: Have %u candidates; user must make selection.".printf(request.candidates.length()));
                     confirm = true;
                 } else {
                     identity = request.candidates.nth_data(0);                    
@@ -240,6 +236,7 @@ public class IdentityManagerApp {
         GLib.Idle.add(
             () => {
                 if (view != null) {
+                    logger.trace("select_identity (Idle handler): calling check_add_password");
                     identity = view.check_add_password(identity, request, model);
                 }
                 request.return_identity(identity);
@@ -253,13 +250,12 @@ public class IdentityManagerApp {
         return;
     }
 
-    private bool match_service_pattern(string service, string pattern)
-        {
-            var pspec = new PatternSpec(pattern);
-            return pspec.match_string(service);
-        }   
+    private bool match_service_pattern(string service, string pattern) {
+        var pspec = new PatternSpec(pattern);
+        return pspec.match_string(service);
+    }   
     
-    #if IPC_MSRPC
+#if IPC_MSRPC
     private void init_ipc_server() {
         // Errors will currently be sent via g_log - ie. to an
         // obtrusive message box, on Windows
@@ -267,7 +263,7 @@ public class IdentityManagerApp {
         this.ipc_server = MoonshotServer.get_instance();
         MoonshotServer.start(this);
     }
-    #elif IPC_DBUS_GLIB
+#elif IPC_DBUS_GLIB
     private void init_ipc_server() {
         try {
             var conn = DBus.Bus.get(DBus.BusType.SESSION);
@@ -280,8 +276,10 @@ public class IdentityManagerApp {
             if (reply == DBus.RequestNameReply.PRIMARY_OWNER)
             {
                 this.ipc_server = new MoonshotServer(this);
+                logger.trace("init_ipc_server(IPC_DBUS_GLIB) : Constructed new MoonshotServer");
                 conn.register_object("/org/janet/moonshot", ipc_server);
             } else {
+                logger.trace("init_ipc_server: reply != PRIMARY_OWNER");
                 bool shown = false;
                 GLib.Error e;
                 DBus.Object manager_proxy = conn.get_object("org.janet.Moonshot",
@@ -300,43 +298,105 @@ public class IdentityManagerApp {
         }
         catch (DBus.Error e)
         {
-            stderr.printf("%s\n", e.message);
+            logger.trace("bus_acquired_cb");
+            try {
+                conn.register_object ("/org/janet/moonshot", ipc_server);
+            }
+            catch (Error e)
+            {
+                stderr.printf ("%s\n", e.message);
+                logger.error("bus_acquired_cb: Caught error: " + e.message);
+            }
         }
     }
-    #else
+#else
     private void bus_acquired_cb(DBusConnection conn) {
+        logger.trace("bus_acquired_cb");
         try {
             conn.register_object("/org/janet/moonshot", ipc_server);
         }
         catch (Error e)
         {
-            stderr.printf("%s\n", e.message);
+            this.ipc_server = new MoonshotServer (this);
+            logger.trace("init_ipc_server: Constructed new MoonshotServer");
+            GLib.Bus.own_name (GLib.BusType.SESSION,
+                               "org.janet.Moonshot",
+                               GLib.BusNameOwnerFlags.NONE,
+                               bus_acquired_cb,
+                               (conn, name) => {logger.trace("init_ipc_server: name_acquired_closure");},
+                               (conn, name) => {
+                                   logger.trace("init_ipc_server: name_lost_closure");
+                                   bool shown=false;
+                                   try {
+                                       IIdentityManager manager = Bus.get_proxy_sync (BusType.SESSION, name, "/org/janet/moonshot");
+                                       shown = manager.show_ui();
+                                   } catch (IOError e) {
+                                       logger.error("init_ipc_server.name_lost_closure: Caught error: ");
+                                   }
+                                   if (!shown) {
+                                       logger.error("init_ipc_server.name_lost_closure: Couldn't own name %s on dbus or show previously launched identity manager".printf(name));
+                                       GLib.error ("Couldn't own name %s on dbus or show previously launched identity manager.", name);
+                                   } else {
+                                       logger.trace("init_ipc_server.name_lost_closure: Showed previously launched identity manager.");
+                                       stdout.printf("Showed previously launched identity manager.\n");
+                                       GLib.Process.exit(0);
+                                   }
+                               });
         }
     }
 
     private void init_ipc_server() {
         this.ipc_server = new MoonshotServer(this);
+        bool shown = false;
         GLib.Bus.own_name(GLib.BusType.SESSION,
                           "org.janet.Moonshot",
                           GLib.BusNameOwnerFlags.NONE,
                           bus_acquired_cb,
-                          (conn, name) => {},
+
+                          // Name acquired callback:
                           (conn, name) => {
-                              bool shown = false;
+                              logger.trace(@"init_ipc_server: name_acquired_closure; show_requested=$show_requested");
+
+                              name_is_owned = true;
+
+                              // Now that we know that we own the name, it's safe to show the UI.
+                              if (show_requested) {
+                                  show();
+                                  show_requested = false;
+                              }
+                              shown = true;
+                          },
+
+                          // Name lost callback:
+                          (conn, name) => {
+                              logger.trace("init_ipc_server: name_lost_closure");
+
+                              // This callback usually means that another moonshot is already running.
+                              // But it *might* mean that we lost the name for some other reason
+                              // (though it's unclear to me yet what those reasons are.)
+                              // Clearing these flags seems like a good idea for that case. -- dbreslau
+                              name_is_owned = false;
+                              show_requested = false;
+
                               try {
-                                  IIdentityManager manager = Bus.get_proxy_sync(BusType.SESSION, name, "/org/janet/moonshot");
-                                  shown = manager.show_ui();
+                                  if (!shown) {
+                                      IIdentityManager manager = Bus.get_proxy_sync(BusType.SESSION, name, "/org/janet/moonshot");
+                                      shown = manager.show_ui();
+                                  }
                               } catch (IOError e) {
+                                  logger.error("init_ipc_server.name_lost_closure: Caught IOError: " + e.message);
                               }
                               if (!shown) {
+                                  logger.error("init_ipc_server.name_lost_closure: Couldn't own name %s on dbus or show previously launched identity manager".printf(name));
                                   GLib.error("Couldn't own name %s on dbus or show previously launched identity manager.", name);
                               } else {
+                                  logger.trace("init_ipc_server.name_lost_closure: Showed previously launched identity manager.");
                                   stdout.printf("Showed previously launched identity manager.\n");
                                   GLib.Process.exit(0);
                               }
                           });
     }
-    #endif
+#endif
 }
 
 static bool explicitly_launched = true;
@@ -351,11 +411,16 @@ const GLib.OptionEntry[] options = {
 
 
 public static int main(string[] args) {
-    #if IPC_MSRPC
-        bool headless = false;
-    #else
-        bool headless = (GLib.Environment.get_variable("DISPLAY") == null);
-    #endif
+
+#if LOG4VALA
+    new IdentityManagerApp.dummy();
+#endif
+
+#if IPC_MSRPC
+    bool headless = false;
+#else
+    bool headless = GLib.Environment.get_variable("DISPLAY") == null;
+#endif
 
     if (headless) {
         try {
@@ -383,29 +448,31 @@ public static int main(string[] args) {
         gtk_available = true;
     }
 
-    #if OS_WIN32
+#if OS_WIN32
     // Force specific theme settings on Windows without requiring a gtkrc file
     Gtk.Settings settings = Gtk.Settings.get_default();
     settings.set_string_property("gtk-theme-name", "ms-windows", "moonshot");
     settings.set_long_property("gtk-menu-images", 0, "moonshot");
-    #endif
+#endif
 
     Intl.bindtextdomain(Config.GETTEXT_PACKAGE, Config.LOCALEDIR);
     Intl.bind_textdomain_codeset(Config.GETTEXT_PACKAGE, "UTF-8");
     Intl.textdomain(Config.GETTEXT_PACKAGE);
        
+       
     var app = new IdentityManagerApp(headless, use_flat_file_store);
     app.explicitly_launched = explicitly_launched;
+    IdentityManagerApp.logger.trace(@"main: explicitly_launched=$explicitly_launched");
         
     if (app.explicitly_launched) {
         app.show();
     }
 
     if (headless) {
-        #if !IPC_MSRPC
+#if !IPC_MSRPC
         MainLoop loop = new MainLoop();
         loop.run();
-        #endif
+#endif
     }
     else {
         Gtk.main();