diff options
Diffstat (limited to 'lib/Plugins')
-rw-r--r-- | lib/Plugins/Bugzilla.cpp | 107 | ||||
-rw-r--r-- | lib/Plugins/Bugzilla.h | 3 | ||||
-rw-r--r-- | lib/Plugins/Catcut.cpp | 3 | ||||
-rw-r--r-- | lib/Plugins/Catcut.h | 1 | ||||
-rw-r--r-- | lib/Plugins/KerneloopsReporter.cpp | 4 | ||||
-rw-r--r-- | lib/Plugins/KerneloopsReporter.h | 1 | ||||
-rw-r--r-- | lib/Plugins/Logger.cpp | 3 | ||||
-rw-r--r-- | lib/Plugins/Logger.h | 1 | ||||
-rw-r--r-- | lib/Plugins/Mailx.cpp | 3 | ||||
-rw-r--r-- | lib/Plugins/Mailx.h | 1 | ||||
-rw-r--r-- | lib/Plugins/TicketUploader.cpp | 3 | ||||
-rw-r--r-- | lib/Plugins/TicketUploader.h | 1 |
12 files changed, 116 insertions, 15 deletions
diff --git a/lib/Plugins/Bugzilla.cpp b/lib/Plugins/Bugzilla.cpp index 84c79347..6a547cf1 100644 --- a/lib/Plugins/Bugzilla.cpp +++ b/lib/Plugins/Bugzilla.cpp @@ -39,6 +39,68 @@ static void get_product_and_version(const std::string& pRelease, std::string& pProduct, std::string& pVersion); +map_plugin_settings_t CReporterBugzilla::parse_settings(const map_plugin_settings_t& pSettings) +{ + map_plugin_settings_t plugin_settings; + map_plugin_settings_t::const_iterator it; + map_plugin_settings_t::const_iterator end = pSettings.end(); + + std::string BugzillaURL; + std::string BugzillaXMLRPC; + + it = pSettings.find("BugzillaURL"); + if (it != end) + { + BugzillaURL = it->second; + //remove the /xmlrpc.cgi part from old settings + //FIXME: can be removed after users are informed about new config format + std::string::size_type pos = BugzillaURL.find(XML_RPC_SUFFIX); + if (pos != std::string::npos) + { + BugzillaURL.erase(pos); + } + //remove the trailing '/' + while (BugzillaURL[BugzillaURL.length() - 1] == '/') + { + BugzillaURL.erase(BugzillaURL.length() - 1); + } + plugin_settings["BugzillaXMLRPC"] = BugzillaURL + XML_RPC_SUFFIX; + plugin_settings["BugzillaURL"] = BugzillaURL; + } + it = pSettings.find("Login"); + if (it != end) + { + plugin_settings["Login"] = it->second; + } + else + { + /* if any of the option is not set we use the defaults for everything */ + plugin_settings.clear(); + return plugin_settings; + } + it = pSettings.find("Password"); + if (it != end) + { + plugin_settings["Password"] = it->second; + } + else + { + plugin_settings.clear(); + return plugin_settings; + } + it = pSettings.find("NoSSLVerify"); + if (it != end) + { + plugin_settings["NoSSLVerify"] = (it->second == "yes"); + } + else + { + plugin_settings.clear(); + return plugin_settings; + } + VERB1 log("User settings ok, using it instead of defaults"); + return plugin_settings; +} // FIXME: we still leak memmory if this function detects a fault: // many instances when we leave non-freed or non-xmlrpc_DECREF'ed data behind. @@ -384,36 +446,61 @@ static void add_attachments(const std::string& pBugId, const map_crash_report_t& } } -std::string CReporterBugzilla::Report(const map_crash_report_t& pCrashReport, const std::string& pArgs) +std::string CReporterBugzilla::Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, + const std::string& pArgs) { int32_t bug_id = -1; - + std::string Login; + std::string Password; + std::string BugzillaXMLRPC; + std::string BugzillaURL; + bool NoSSLVerify; + map_plugin_settings_t settings = parse_settings(pSettings); + /* if parse_settings fails it returns an empty map so we need to use defaults*/ + if(!settings.empty()) + { + Login = settings["Login"]; + Password = settings["Password"]; + BugzillaXMLRPC = settings["BugzillaXMLRPC"]; + BugzillaURL = settings["BugzillaURL"]; + NoSSLVerify = settings["NoSSLVerify"] == "yes"; + } + else + { + Login = m_sLogin; + Password = m_sPassword; + BugzillaXMLRPC = m_sBugzillaXMLRPC; + BugzillaURL = m_sBugzillaURL; + NoSSLVerify = m_bNoSSLVerify; + } + std::string component = pCrashReport.find(FILENAME_COMPONENT)->second[CD_CONTENT]; std::string uuid = pCrashReport.find(CD_UUID)->second[CD_CONTENT]; try { - new_xmlrpc_client(m_sBugzillaXMLRPC.c_str(), m_bNoSSLVerify); + new_xmlrpc_client(BugzillaXMLRPC.c_str(), NoSSLVerify); update_client(_("Checking for duplicates...")); bug_id = check_uuid_in_bugzilla(component.c_str(), uuid.c_str()); update_client(_("Logging into bugzilla...")); - if ((m_sLogin == "") && (m_sPassword == "")) + if ((Login == "") && (Password == "")) { VERB3 log("Empty login and password"); throw CABRTException(EXCEP_PLUGIN, std::string(_("Empty login and password. Please check Bugzilla.conf"))); } - login(m_sLogin.c_str(), m_sPassword.c_str()); + login(Login.c_str(), Password.c_str()); if (bug_id > 0) { update_client(_("Checking CC...")); - if (!check_cc_and_reporter(bug_id, m_sLogin.c_str())) + if (!check_cc_and_reporter(bug_id, Login.c_str())) { - add_plus_one_cc(bug_id, m_sLogin.c_str()); + add_plus_one_cc(bug_id, Login.c_str()); } destroy_xmlrpc_client(); - return m_sBugzillaURL + "/show_bug.cgi?id="+to_string(bug_id); + return BugzillaURL + "/show_bug.cgi?id="+to_string(bug_id); } update_client(_("Creating new bug...")); @@ -433,10 +520,10 @@ std::string CReporterBugzilla::Report(const map_crash_report_t& pCrashReport, co if (bug_id > 0) { - return m_sBugzillaURL + "/show_bug.cgi?id="+to_string(bug_id); + return BugzillaURL + "/show_bug.cgi?id="+to_string(bug_id); } - return m_sBugzillaURL + "/show_bug.cgi?id="; + return BugzillaURL + "/show_bug.cgi?id="; } void CReporterBugzilla::SetSettings(const map_plugin_settings_t& pSettings) diff --git a/lib/Plugins/Bugzilla.h b/lib/Plugins/Bugzilla.h index 17864650..702eead2 100644 --- a/lib/Plugins/Bugzilla.h +++ b/lib/Plugins/Bugzilla.h @@ -14,6 +14,8 @@ class CReporterBugzilla : public CReporter std::string m_sLogin; std::string m_sPassword; std::string m_sAttchmentInBase64; + + map_plugin_settings_t parse_settings(const map_plugin_settings_t& pSettings); public: CReporterBugzilla(); @@ -21,6 +23,7 @@ class CReporterBugzilla : public CReporter virtual void SetSettings(const map_plugin_settings_t& pSettings); virtual map_plugin_settings_t GetSettings(); virtual std::string Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs); }; diff --git a/lib/Plugins/Catcut.cpp b/lib/Plugins/Catcut.cpp index c833a577..5f0583f4 100644 --- a/lib/Plugins/Catcut.cpp +++ b/lib/Plugins/Catcut.cpp @@ -298,7 +298,8 @@ CReporterCatcut::CReporterCatcut() : CReporterCatcut::~CReporterCatcut() {} -string CReporterCatcut::Report(const map_crash_report_t& pCrashReport, const string& pArgs) +string CReporterCatcut::Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const string& pArgs) { update_client(_("Creating new bug...")); try diff --git a/lib/Plugins/Catcut.h b/lib/Plugins/Catcut.h index 69380387..40ef399d 100644 --- a/lib/Plugins/Catcut.h +++ b/lib/Plugins/Catcut.h @@ -20,6 +20,7 @@ class CReporterCatcut : public CReporter virtual map_plugin_settings_t GetSettings(); virtual std::string Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs); }; diff --git a/lib/Plugins/KerneloopsReporter.cpp b/lib/Plugins/KerneloopsReporter.cpp index 8344b810..c57a1ef8 100644 --- a/lib/Plugins/KerneloopsReporter.cpp +++ b/lib/Plugins/KerneloopsReporter.cpp @@ -93,7 +93,9 @@ CKerneloopsReporter::CKerneloopsReporter() : m_sSubmitURL("http://submit.kerneloops.org/submitoops.php") {} -std::string CKerneloopsReporter::Report(const map_crash_report_t& pCrashReport, const std::string& pArgs) +std::string CKerneloopsReporter::Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, + const std::string& pArgs) { int ret = -1; map_crash_report_t::const_iterator it; diff --git a/lib/Plugins/KerneloopsReporter.h b/lib/Plugins/KerneloopsReporter.h index f1b427f8..3c165920 100644 --- a/lib/Plugins/KerneloopsReporter.h +++ b/lib/Plugins/KerneloopsReporter.h @@ -43,6 +43,7 @@ class CKerneloopsReporter : public CReporter virtual void SetSettings(const map_plugin_settings_t& pSettings); virtual map_plugin_settings_t GetSettings(); virtual std::string Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs); }; diff --git a/lib/Plugins/Logger.cpp b/lib/Plugins/Logger.cpp index eb9c2404..81787940 100644 --- a/lib/Plugins/Logger.cpp +++ b/lib/Plugins/Logger.cpp @@ -53,7 +53,8 @@ map_plugin_settings_t CLogger::GetSettings() return ret; } -std::string CLogger::Report(const map_crash_report_t& pCrashReport, const std::string& pArgs) +std::string CLogger::Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs) { update_client(_("Creating a report...")); diff --git a/lib/Plugins/Logger.h b/lib/Plugins/Logger.h index 60c76e4a..33f957db 100644 --- a/lib/Plugins/Logger.h +++ b/lib/Plugins/Logger.h @@ -37,6 +37,7 @@ class CLogger : public CReporter virtual void SetSettings(const map_plugin_settings_t& pSettings); virtual map_plugin_settings_t GetSettings(); virtual std::string Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs); }; diff --git a/lib/Plugins/Mailx.cpp b/lib/Plugins/Mailx.cpp index f7a86fd0..ef3a8d2a 100644 --- a/lib/Plugins/Mailx.cpp +++ b/lib/Plugins/Mailx.cpp @@ -118,7 +118,8 @@ void CMailx::SendEmail(const std::string& pSubject, const std::string& pText, co ExecMailx(atoi(pUID.c_str()), pText); } -std::string CMailx::Report(const map_crash_report_t& pCrashReport, const std::string& pArgs) +std::string CMailx::Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs) { update_client(_("Creating a report...")); diff --git a/lib/Plugins/Mailx.h b/lib/Plugins/Mailx.h index fd00bb8b..e3c86e73 100644 --- a/lib/Plugins/Mailx.h +++ b/lib/Plugins/Mailx.h @@ -49,6 +49,7 @@ class CMailx : public CReporter virtual void SetSettings(const map_plugin_settings_t& pSettings); virtual map_plugin_settings_t GetSettings(); virtual std::string Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs); }; diff --git a/lib/Plugins/TicketUploader.cpp b/lib/Plugins/TicketUploader.cpp index 69243cd9..179ccc07 100644 --- a/lib/Plugins/TicketUploader.cpp +++ b/lib/Plugins/TicketUploader.cpp @@ -218,7 +218,8 @@ void CTicketUploader::SendFile(const std::string& pURL, } -string CTicketUploader::Report(const map_crash_report_t& pCrashReport, const std::string& pArgs) +string CTicketUploader::Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs) { string ret; update_client(_("Creating an TicketUploader report...")); diff --git a/lib/Plugins/TicketUploader.h b/lib/Plugins/TicketUploader.h index 68a29638..92e33872 100644 --- a/lib/Plugins/TicketUploader.h +++ b/lib/Plugins/TicketUploader.h @@ -65,6 +65,7 @@ class CTicketUploader : public CReporter virtual void SetSettings(const map_plugin_settings_t& pSettings); virtual string Report(const map_crash_report_t& pCrashReport, + const map_plugin_settings_t& pSettings, const std::string& pArgs); |