[epiphany/wip/sync] profile-migrator: Migrate passwords to Firefox Sync passwords
- From: Gabriel Ivașcu <gabrielivascu src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [epiphany/wip/sync] profile-migrator: Migrate passwords to Firefox Sync passwords
- Date: Wed, 3 May 2017 23:01:02 +0000 (UTC)
commit 9ca20425820d57c691268465cf1ee0261c316953
Author: Gabriel Ivascu <ivascu gabriel59 gmail com>
Date: Wed May 3 22:42:42 2017 +0300
profile-migrator: Migrate passwords to Firefox Sync passwords
lib/ephy-profile-utils.h | 3 +-
src/profile-migrator/ephy-profile-migrator.c | 73 ++++++++++++++++++++++++++
2 files changed, 75 insertions(+), 1 deletions(-)
---
diff --git a/lib/ephy-profile-utils.h b/lib/ephy-profile-utils.h
index dd8943d..ac4841d 100644
--- a/lib/ephy-profile-utils.h
+++ b/lib/ephy-profile-utils.h
@@ -24,9 +24,10 @@
G_BEGIN_DECLS
-#define EPHY_PROFILE_MIGRATION_VERSION 17
+#define EPHY_PROFILE_MIGRATION_VERSION 18
#define EPHY_INSECURE_PASSWORDS_MIGRATION_VERSION 11
#define EPHY_SETTINGS_MIGRATION_VERSION 16
+#define EPHY_FIREFOX_SYNC_PASSWORDS_MIGRATION_VERSION 18
#define EPHY_BOOKMARKS_FILE "bookmarks.gvdb"
#define EPHY_HISTORY_FILE "ephy-history.db"
diff --git a/src/profile-migrator/ephy-profile-migrator.c b/src/profile-migrator/ephy-profile-migrator.c
index 42a65c3..5bc9c7f 100644
--- a/src/profile-migrator/ephy-profile-migrator.c
+++ b/src/profile-migrator/ephy-profile-migrator.c
@@ -975,6 +975,78 @@ migrate_search_engines (void)
}
static void
+migrate_passwords_to_firefox_sync_passwords (void)
+{
+ GHashTable *attributes;
+ GList *passwords = NULL;
+ GError *error = NULL;
+ int default_profile_migration_version;
+
+ /* Similar to the insecure passowrds migration, we want to migrate passwords
+ * to Firefox Sync passwords only once since saved passwords are stored
+ * globally and not per profile. This won't affect password lookup for web
+ * apps because this migration only adds a couple of new fields to the
+ * password schema, fields that are not taken into consideration when
+ * querying passwords.
+ */
+ default_profile_migration_version = ephy_profile_utils_get_migration_version_for_profile_dir
(ephy_default_dot_dir ());
+ if (default_profile_migration_version >= EPHY_FIREFOX_SYNC_PASSWORDS_MIGRATION_VERSION)
+ return;
+
+ attributes = secret_attributes_build (EPHY_FORM_PASSWORD_SCHEMA, NULL);
+ passwords = secret_service_search_sync (NULL, EPHY_FORM_PASSWORD_SCHEMA, attributes,
+ SECRET_SEARCH_ALL | SECRET_SEARCH_UNLOCK |
SECRET_SEARCH_LOAD_SECRETS,
+ NULL, &error);
+ if (error) {
+ g_warning ("Failed to search the password schema: %s", error->message);
+ goto out;
+ }
+
+ secret_service_clear_sync (NULL, EPHY_FORM_PASSWORD_SCHEMA,
+ attributes, NULL, &error);
+ if (error) {
+ g_warning ("Failed to clear the password schema: %s", error->message);
+ goto out;
+ }
+
+ for (GList *l = passwords; l && l->data; l = l->next) {
+ SecretItem *item = (SecretItem *)l->data;
+ SecretValue *value = secret_item_get_secret (item);
+ GHashTable *attrs = secret_item_get_attributes (item);
+ const char *hostname = g_hash_table_lookup (attrs, HOSTNAME_KEY);
+ const char *username = g_hash_table_lookup (attrs, USERNAME_KEY);
+ char *uuid = g_uuid_string_random ();
+ char *label;
+
+ g_hash_table_insert (attrs, g_strdup (ID_KEY), g_strdup_printf ("{%s}", uuid));
+ g_hash_table_insert (attrs, g_strdup (SERVER_TIME_MODIFIED_KEY), g_strdup ("0"));
+
+ if (username)
+ label = g_strdup_printf ("Password for %s in a form in %s", username, hostname);
+ else
+ label = g_strdup_printf ("Password in a form in %s", hostname);
+ secret_service_store_sync (NULL, EPHY_FORM_PASSWORD_SCHEMA,
+ attrs, NULL, label,
+ value, NULL, &error);
+ if (error) {
+ g_warning ("Failed to store password: %s", error->message);
+ g_clear_pointer (&error, g_error_free);
+ }
+
+ g_free (label);
+ g_free (uuid);
+ secret_value_unref (value);
+ g_hash_table_unref (attrs);
+ }
+
+out:
+ if (error)
+ g_error_free (error);
+ g_hash_table_unref (attributes);
+ g_list_free_full (passwords, g_object_unref);
+}
+
+static void
migrate_nothing (void)
{
/* Used to replace migrators that have been removed. Only remove migrators
@@ -1005,6 +1077,7 @@ const EphyProfileMigrator migrators[] = {
/* 15 */ migrate_permissions,
/* 16 */ migrate_settings,
/* 17 */ migrate_search_engines,
+ /* 18 */ migrate_passwords_to_firefox_sync_passwords,
};
static gboolean
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]