DEV: Stop logging theme load errors to logster (#36222)

davidtaylorhq · web-flow · commit 186883998d54 · 2025-11-25T14:53:50.000Z
In our experience, this is creating a significant amount of
un-actionable noise in the logs. Themes can fail to boot for all sorts
of client-caused reasons, like a poor network connection, adblocker,
unsupported browser, etc.

Logging those to Logster is not helpful, and only leads to confusion for
admins and hosting providers.
diff --git a/config/discourse_defaults.conf b/config/discourse_defaults.conf
@@ -324,7 +324,7 @@ maxmind_mirror_url =
 enable_performance_http_headers = false
 
 # gather JavaScript errors from clients (rate limited to 1 error per IP per minute)
-enable_js_error_reporting = true
+enable_js_error_reporting = false
 
 # This is probably not a number you want to touch, it controls the number of workers
 # we allow mini scheduler to run. Prior to 2019 we ran a single worker.
diff --git a/frontend/discourse/app/services/client-error-handler.js b/frontend/discourse/app/services/client-error-handler.js
@@ -1,10 +1,8 @@
 import { getOwner } from "@ember/owner";
 import Service, { service } from "@ember/service";
-import $ from "jquery";
 import { getAndClearUnhandledThemeErrors } from "discourse/app";
 import { bind } from "discourse/lib/decorators";
 import escape from "discourse/lib/escape";
-import getURL from "discourse/lib/get-url";
 import { disableImplicitInjections } from "discourse/lib/implicit-injections";
 import identifySource, {
   consolePrefix,
@@ -56,7 +54,6 @@ export default class ClientErrorHandlerService extends Service {
     };
 
     reportToConsole(error, source);
-    reportToLogster(source.name, error);
 
     const message = i18n("themes.broken_theme_alert");
     this.displayErrorNotice(message, source);
@@ -105,19 +102,6 @@ export default class ClientErrorHandlerService extends Service {
   }
 }
 
-function reportToLogster(name, error) {
-  const data = {
-    message: `${name} theme/component is throwing errors:\n${error.name}: ${error.message}`,
-    stacktrace: error.stack,
-  };
-
-  // TODO: To be moved out into a logster-provided lib
-  $.ajax(getURL("/logs/report_js_error"), {
-    data,
-    type: "POST",
-  });
-}
-
 function reportToConsole(error, source) {
   const prefix = consolePrefix(error, source);
   if (prefix) {
