From e2e4c246fad876f129e724ab95d695cae153707e Mon Sep 17 00:00:00 2001 From: Carl Corcoran Date: Mon, 28 Aug 2017 08:43:24 +0200 Subject: [PATCH] bug 1380254: Don't send StackTraces field to Socorro; r=gsvelto MozReview-Commit-ID: 3K5I5ZQam7q --HG-- extra : rebase_source : 9a089624c8656a136c16f24c3bf9cc002f3f131a --- toolkit/crashreporter/CrashSubmit.jsm | 2 +- toolkit/crashreporter/client/crashreporter.cpp | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/toolkit/crashreporter/CrashSubmit.jsm b/toolkit/crashreporter/CrashSubmit.jsm index fde213a96933..187a6ca6524b 100644 --- a/toolkit/crashreporter/CrashSubmit.jsm +++ b/toolkit/crashreporter/CrashSubmit.jsm @@ -227,7 +227,7 @@ Submitter.prototype = { .createInstance(Ci.nsIDOMFormData); // add the data for (let [name, value] of Object.entries(this.extraKeyVals)) { - if (name != "ServerURL") { + if (name != "ServerURL" && name != "StackTraces") { formData.append(name, value); } } diff --git a/toolkit/crashreporter/client/crashreporter.cpp b/toolkit/crashreporter/client/crashreporter.cpp index de66f5e8f52f..69cb6b98cb4e 100644 --- a/toolkit/crashreporter/client/crashreporter.cpp +++ b/toolkit/crashreporter/client/crashreporter.cpp @@ -757,8 +757,9 @@ int main(int argc, char** argv) } string sendURL = queryParameters["ServerURL"]; - // we don't need to actually send this + // we don't need to actually send these queryParameters.erase("ServerURL"); + queryParameters.erase("StackTraces"); queryParameters["Throttleable"] = "1";