From 65d7c6231130f7448f537449b9267285810510d4 Mon Sep 17 00:00:00 2001 From: Vincent Zanotti Date: Sat, 25 Oct 2008 23:06:36 +0200 Subject: [PATCH] Adds support of plat/al's aliases in Google Apps. It will solve the issue of the "Sender:" header being added to outgoing mails on GoogleApps GMail. Signed-off-by: Vincent Zanotti --- bin/cron/google_apps.php | 66 +++++++++++++++++++++++++++++ upgrade/0.9.17/08_google_apps_nicknames.sql | 21 +++++++++ 2 files changed, 87 insertions(+) create mode 100644 upgrade/0.9.17/08_google_apps_nicknames.sql diff --git a/bin/cron/google_apps.php b/bin/cron/google_apps.php index 50212ae..925cb33 100755 --- a/bin/cron/google_apps.php +++ b/bin/cron/google_apps.php @@ -56,6 +56,72 @@ while ($account = $res->next()) { } } +/* Updates the l_userid parameter for newer nicknames. */ +$res = XDB::iterator( + "SELECT g.g_account_name, a.id + FROM gapps_nicknames AS g + LEFT JOIN aliases AS a ON (a.alias = g.g_account_name AND a.type = 'a_vie') + WHERE (g.l_userid IS NULL or g.l_userid <= 0) AND a.id IS NOT NULL + GROUP BY g_account_name"); +while ($nickname = $res->next()) { + XDB::execute( + "UPDATE gapps_nicknames + SET l_userid = {?} + WHERE g_account_name = {?}", + $nickname['id'], $nickname['g_account_name']); +} + +/* Emits a warning for nicknames without local user_id. */ +$res = XDB::iterator( + "SELECT g.g_account_name + FROM gapps_nicknames AS g + LEFT JOIN aliases as a ON (a.alias = g.g_account_name AND a.type = 'a_vie') + WHERE (g.l_userid IS NULL OR g.l_userid <= 0) AND a.id IS NULL"); +while ($nickname = $res->next()) { + if (!preg_match("/^admin-/", $nickname['g_account_name'])) { + printf("Warning: Nickname '%s' has no local user_id.\n", $nickname['g_account_name']); + } +} + +/* Checks that all nicknames have been synchronized to GoogleApps. Creates the + missing ones. */ +$res = XDB::iterator( + "SELECT g.l_userid AS id, f.alias AS username, a.alias AS nickname + FROM gapps_accounts AS g + INNER JOIN aliases AS f ON (f.id = g.l_userid AND f.type = 'a_vie') + INNER JOIN aliases AS a ON (a.id = g.l_userid AND a.type = 'alias') + LEFT JOIN gapps_nicknames AS n ON (n.l_userid = g.l_userid AND n.g_nickname = a.alias) + WHERE n.g_nickname IS NULL AND g.l_userid IS NOT NULL"); +while ($nickname = $res->next()) { + // Checks that the requested nickname doesn't look like a regular forlife; + // we might run in troubler later if we don't keep the two repos. If we need + // to add a forlife-looking nickname at some point, we'll do it manually. + if (!preg_match('/^[-a-z]+\.[-a-z]+\.\d{4}$/', $nickname['nickname'])) { + XDB::execute( + "INSERT INTO gapps_queue + SET q_recipient_id = {?}, p_entry_date = NOW(), p_notbefore_date = NOW(), + p_priority = 'offline', j_type = 'n_create', j_parameters = {?}", + $nickname['id'], + json_encode($nickname)); + } +} + +/* Checks that all nicknames in GoogleApps are also aliases on plat/al side. + Deletes the invalid ones. */ +$res = XDB::iterator( + "SELECT g.l_userid AS id, g.g_nickname AS nickname + FROM gapps_nicknames AS g + LEFT JOIN aliases AS a ON (a.id = g.l_userid AND a.type = 'alias' AND a.alias = g.g_nickname) + WHERE g.l_userid IS NOT NULL AND a.alias IS NULL"); +while ($nickname = $res->next()) { + XDB::execute( + "INSERT INTO gapps_queue + SET q_recipient_id = {?}, p_entry_date = NOW(), p_notbefore_date = NOW(), + p_priority = 'offline', j_type = 'n_delete', j_parameters = {?}", + $nickname['id'], + json_encode($nickname)); +} + /* Retrieves successful job queues for post-queue processing. */ $res = XDB::iterator( "SELECT q_id, q_recipient_id, j_type, j_parameters diff --git a/upgrade/0.9.17/08_google_apps_nicknames.sql b/upgrade/0.9.17/08_google_apps_nicknames.sql new file mode 100644 index 0000000..bb1541c --- /dev/null +++ b/upgrade/0.9.17/08_google_apps_nicknames.sql @@ -0,0 +1,21 @@ +-- This SQL table comes from the 'gapps-daemon' project. + +-- Table `gapps_nicknames` +-- Holds the list of all nicknames registered in the Google Apps domain. +CREATE TABLE IF NOT EXISTS `gapps_nicknames` ( + -- Application-specific field. + l_userid INT DEFAULT NULL, + + -- Shared fields. + g_account_name VARCHAR(256) NOT NULL, + g_nickname VARCHAR(256) NOT NULL, + -- Indexes. + PRIMARY KEY(g_nickname), + INDEX g_account_name(g_account_name), + INDEX l_userid(l_userid) +) CHARSET=utf8; + +-- Add new task types. +ALTER TABLE gapps_queue MODIFY j_type ENUM('n_create', 'n_delete', 'n_resync', 'r_activity', 'r_accounts', 'u_create', 'u_delete', 'u_update', 'u_sync') NOT NULL; + +-- vim:set syntax=mysql: -- 2.1.4