doczkal / rpms / abrt

Forked from rpms/abrt 4 years ago
Clone
Blob Blame History Raw
From cafb9545c0e251b8ece493a341e7b209bde00a11 Mon Sep 17 00:00:00 2001
From: Martin Milata <mmilata@redhat.com>
Date: Mon, 20 Jan 2014 15:24:31 +0100
Subject: [ABRT PATCH 13/13] applet: do not say the report is anonymous when
 ureport auth is enabled

Depends on https://github.com/abrt/libreport/pull/232

Related to rhbz#1053042, rhbz#1055619.

Signed-off-by: Martin Milata <mmilata@redhat.com>
---
 configure.ac           |  2 ++
 src/applet/Makefile.am |  1 +
 src/applet/applet.c    | 43 +++++++++++++++++++++++++++++++++++++++----
 3 files changed, 42 insertions(+), 4 deletions(-)

diff --git a/configure.ac b/configure.ac
index a5f7a79..479289c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -139,6 +139,7 @@ EVENTS_DIR='${datadir}/libreport/events'
 EVENTS_CONF_DIR='${sysconfdir}/libreport/events.d'
 ENABLE_SOCKET_OR_DBUS='-DENABLE_DBUS=1'
 DEFAULT_DUMP_DIR_MODE=$($PKG_CONFIG --variable=dd_mode libreport)
+LIBREPORT_PLUGINS_CONF_DIR=$($PKG_CONFIG --variable=plugins_conf_dir libreport)
 PROBLEMS_CONFIG_INTERFACES_DIR=${dbusinterfacedir}
 
 AC_ARG_WITH([defaultdumplocation],
@@ -226,6 +227,7 @@ AC_SUBST(EVENTS_CONF_DIR)
 AC_SUBST(EVENTS_DIR)
 AC_SUBST(DEFAULT_DUMP_LOCATION)
 AC_SUBST(DEFAULT_DUMP_DIR_MODE)
+AC_SUBST(LIBREPORT_PLUGINS_CONF_DIR)
 AC_SUBST(PROBLEMS_CONFIG_INTERFACES_DIR)
 
 AC_ARG_WITH(bodhi,
diff --git a/src/applet/Makefile.am b/src/applet/Makefile.am
index ec50fcd..fe7ec98 100644
--- a/src/applet/Makefile.am
+++ b/src/applet/Makefile.am
@@ -9,6 +9,7 @@ abrt_applet_CPPFLAGS = \
     -DBIN_DIR=\"$(bindir)\" \
     -DLIBEXEC_DIR=\"$(libexecdir)\" \
     -DICON_DIR=\"${datadir}/abrt/icons/hicolor/48x48/status\" \
+    -DLIBREPORT_PLUGINS_CONF_DIR=\"$(LIBREPORT_PLUGINS_CONF_DIR)\" \
     $(XICE_CFLAGS) \
     $(XSMP_CFLAGS) \
     $(GTK_CFLAGS) \
diff --git a/src/applet/applet.c b/src/applet/applet.c
index 9971e04..131d2ab 100644
--- a/src/applet/applet.c
+++ b/src/applet/applet.c
@@ -90,6 +90,27 @@ static bool is_autoreporting_enabled(void)
     return get_configured_bool_or_default("AutoreportingEnabled", g_settings_autoreporting);
 }
 
+static bool is_ureport_auth_enabled(void)
+{
+    bool success, auth_enabled;
+    map_string_t *settings = new_map_string();
+    char *ureport_conf_path = concat_path_file(LIBREPORT_PLUGINS_CONF_DIR, "ureport.conf");
+
+    success = load_conf_file(ureport_conf_path, settings, /*skipKeysWithoutValue*/false);
+    if (success)
+    {
+        const char *value = get_map_string_item_or_NULL(settings, "SSLClientAuth");
+        auth_enabled = (value && value[0] != '\0');
+    }
+    else
+        auth_enabled = true; /* assume it is, do not claim the reporting is anonymous */
+
+    free(ureport_conf_path);
+    free_map_string(settings);
+
+    return auth_enabled;
+}
+
 static const char *get_autoreport_event_name(void)
 {
     load_user_settings("abrt-applet");
@@ -99,14 +120,28 @@ static const char *get_autoreport_event_name(void)
 
 static void ask_start_autoreporting()
 {
+    struct strbuf *question = strbuf_new();
+    question = strbuf_append_str(question,
+         _("The report which will be sent does not contain any security sensitive data. "
+           "Therefore it is not necessary to bother you next time and require any further action by you. \n"));
+
+    if (is_ureport_auth_enabled())
+    {
+        question = strbuf_append_str(question,
+            _("Do you want to enable automatically submitted crash reports?"));
+    }
+    else
+    {
+        question = strbuf_append_str(question,
+            _("Do you want to enable automatically submitted anonymous crash reports?"));
+    }
+
     /* The "Yes" response will be saved even if user don't check the
      * "Don't ask me again" box.
      */
-    const int ret = run_ask_yes_no_save_result_dialog("AutoreportingEnabled",
-     _("The report which will be sent does not contain any security sensitive data. "
-       "Therefore it is not necessary to bother you next time and require any further action by you. "
-       "\nDo you want to enable automatically submitted anonymous crash reports?"),
+    const int ret = run_ask_yes_no_save_result_dialog("AutoreportingEnabled", question->buf,
        /*parent wnd */ NULL);
+    strbuf_free(question);
 
     load_user_settings("abrt-applet");
 
-- 
1.8.3.1