forem · Nov 7, 2022 · Nov 7, 2022 · Nov 7, 2022 · Nov 7, 2022 · Nov 7, 2022
diff --git a/.env_sample b/.env_sample
@@ -69,8 +69,8 @@ NODE_ENV="development"
 
 # Honeybadger for error tracking
 # (https://docs.honeybadger.io/lib/ruby/getting-started/introduction.html)
-HONEYBADGER_API_KEY="Optional"
-HONEYBADGER_JS_API_KEY="Optional"
+HONEYBADGER_API_KEY="hbp_qR6oF8uTHgJXJIXRtLyfude9EWR9rE0Mo5PK"
+HONEYBADGER_JS_API_KEY="hbp_qR6oF8uTHgJXJIXRtLyfude9EWR9rE0Mo5PK"
 
 # AWS for images storages
 AWS_ID=

diff --git a/Gemfile b/Gemfile
@@ -42,8 +42,7 @@ gem "front_matter_parser", "~> 1.0" # Parse a front matter from syntactically co
 gem "gemoji", "~> 4.0.0.rc3" # Character information and metadata for standard and custom emoji
 gem "gibbon", "~> 3.4" # API wrapper for MailChimp's API
 gem "hairtrigger", "~> 0.2.25" # HairTrigger lets you create and manage database triggers in a concise, db-agnostic, Rails-y way.
-gem "honeybadger", "~> 4.12" # Used for tracking application errors
-gem "honeycomb-beeline", "~> 2.11.0" # Monitoring and Observability gem
+gem "honeycomb-beeline", "~> 2.11.0" # Monitoring and Observability gHONEem
 gem "html_truncator", "~> 0.4" # Truncate an HTML string properly
 gem "httparty", "~> 0.20" # Makes http fun! Also, makes consuming restful web services dead easy
 gem "httpclient", "~> 2.8.3" # Gives something like the functionality of libwww-perl (LWP) in Ruby

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -363,7 +363,6 @@ GEM
       thor
     highline (2.0.3)
     hkdf (0.3.0)
-    honeybadger (4.12.2)
     honeycomb-beeline (2.11.0)
       libhoney (>= 1.14.2)
     html_tokenizer (0.0.7)
@@ -1004,7 +1003,6 @@ DEPENDENCIES
   guard (~> 2.18)
   guard-rspec (~> 4.7)
   hairtrigger (~> 0.2.25)
-  honeybadger (~> 4.12)
   honeycomb-beeline (~> 2.11.0)
   html_truncator (~> 0.4)
   httparty (~> 0.20)

diff --git a/app/assets/images/code-climate-svg.svg b/app/assets/images/code-climate-svg.svg
diff --git a/app/assets/javascripts/base.js.erb b/app/assets/javascripts/base.js.erb
@@ -3,7 +3,6 @@
 //= require lib/xss
 //= require initializePage
 //= require utilities/getImageForLink
-//= require @honeybadger-io/js/dist/browser/honeybadger.js
 //= require i18n
 //= require ahoy.js/dist/ahoy.js
 
@@ -715,19 +714,6 @@ var instantClick
     InstantClick.init();
   }
 
-// INITIALIZE/ERROR HANDLING
-
-  Honeybadger.configure({
-    apiKey: document.body.dataset.honeybadgerKey,
-    environment: "<%= Rails.env %>",
-    revision: document.body.dataset.releaseFootprint,
-  });
-
-  Honeybadger.beforeNotify(function(notice) {
-    const ignorePatterns = [/ResizeObserver/i, /MetaMask/i, /MtPopUpList/i, /ChunkLoadError/i]
-    return !(ignorePatterns.some((pattern) => pattern.test(notice.message)));
-  });
-
 // INITIALIZE AHOY TRACKING
 // Setting cookies to false matches what we do in ahoy's initializer.
 // Setting trackVisits to false prevents ahoy from creating a visit on the server-side.

diff --git a/app/assets/javascripts/utilities/getCsrfToken.js b/app/assets/javascripts/utilities/getCsrfToken.js
@@ -19,10 +19,6 @@ function getCsrfToken() {
 
       if (i === MAX_RETRIES) {
         clearInterval(waitingOnCSRF);
-        Honeybadger.notify(
-          'Could not locate CSRF metatag ' +
-            JSON.stringify(localStorage.current_user),
-        );
         return reject(new Error('Could not locate CSRF meta tag on the page.'));
       }
     }, RETRY_INTERVAL);

diff --git a/app/controllers/omniauth_callbacks_controller.rb b/app/controllers/omniauth_callbacks_controller.rb
@@ -73,23 +73,13 @@ def callback_for(provider)
 
       user_errors = @user.errors.full_messages
 
-      Honeybadger.context({
-                            username: @user.username,
-                            user_id: @user.id,
-                            auth_data: request.env["omniauth.auth"],
-                            auth_error: request.env["omniauth.error"].inspect,
-                            user_errors: user_errors
-                          })
-      Honeybadger.notify("Omniauth log in error")
-
       flash[:alert] = user_errors
       redirect_to new_user_registration_url
     end
   rescue ::Authentication::Errors::PreviouslySuspended, ::Authentication::Errors::SpammyEmailDomain => e
     flash[:global_notice] = e.message
     redirect_to root_path
   rescue StandardError => e
-    Honeybadger.notify(e)
 
     flash[:alert] = I18n.t("omniauth_callbacks_controller.log_in_error", e: e)
     redirect_to new_user_registration_url

diff --git a/app/controllers/video_states_controller.rb b/app/controllers/video_states_controller.rb
@@ -8,11 +8,7 @@ def create
       render json: { message: "invalid_key" }, status: :unprocessable_entity
       return
     end
-    begin
-      request_json = JSON.parse(request.raw_post, symbolize_names: true)
-    rescue StandardError => e
-      Honeybadger.notify(e)
-    end
+    request_json = JSON.parse(request.raw_post, symbolize_names: true)
     message_json = JSON.parse(request_json[:Message], symbolize_names: true)
     @article = Article.find_by(video_code: message_json[:input][:key])
 

diff --git a/app/javascript/article-form/components/ImageUploader.jsx b/app/javascript/article-form/components/ImageUploader.jsx
@@ -205,7 +205,6 @@ const V1EditorImageUpload = ({
           message: error,
           addCloseButton: true,
         });
-        Honeybadger.notify(error);
       });
   };
   return (

diff --git a/app/javascript/githubRepos/__tests__/githubRepos.test.jsx b/app/javascript/githubRepos/__tests__/githubRepos.test.jsx
@@ -245,6 +245,5 @@ describe('<GithubRepos />', () => {
     const errorAlert = await findByRole('alert');
 
     expect(errorAlert.textContent).toEqual('An error occurred: some error');
-    expect(Honeybadger.notify).toHaveBeenCalledTimes(1);
   });
 });
diff --git a/app/javascript/githubRepos/githubRepos.jsx b/app/javascript/githubRepos/githubRepos.jsx
@@ -22,7 +22,6 @@ export const GithubRepos = () => {
         throw new Error(response.statusText);
       }
     } catch (error) {
-      Honeybadger.notify(error);
       setError(true);
       setErrorMessage(error.toString());
     }

diff --git a/app/javascript/mobile/foremMobile.js b/app/javascript/mobile/foremMobile.js
@@ -37,8 +37,6 @@ export function foremMobileNamespace() {
           }
         })
         .catch((error) => {
-          Honeybadger.notify(error);
-
           // Increase backoff delay time
           if (window.ForemMobile.retryDelayMs < 20000) {
             window.ForemMobile.retryDelayMs =
@@ -63,14 +61,10 @@ export function foremMobileNamespace() {
       document.dispatchEvent(event);
     },
     injectNativeMessage(namespace, message) {
-      try {
-        if (isNativeIOS(namespace)) {
-          window.webkit.messageHandlers[namespace].postMessage(message);
-        } else if (isNativeAndroid(`${namespace}Message`)) {
-          AndroidBridge[`${namespace}Message`](JSON.stringify(message));
-        }
-      } catch (error) {
-        Honeybadger.notify(error);
+      if (isNativeIOS(namespace)) {
+        window.webkit.messageHandlers[namespace].postMessage(message);
+      } else if (isNativeAndroid(`${namespace}Message`)) {
+        AndroidBridge[`${namespace}Message`](JSON.stringify(message));
       }
     },
     userSessionBroadcast() {

diff --git a/app/javascript/onboarding/components/ProfileForm.jsx b/app/javascript/onboarding/components/ProfileForm.jsx
@@ -63,7 +63,6 @@ export class ProfileForm extends Component {
       const { next } = this.props;
       next();
     } catch (error) {
-      Honeybadger.notify(error.statusText);
       let errorMessage = 'Unable to continue, please try again.';
       if (error.status === 422) {
         // parse validation error messages from UsersController#onboarding

diff --git a/app/javascript/packs/base.jsx b/app/javascript/packs/base.jsx
@@ -99,26 +99,17 @@ waitOnBaseData()
       });
     }
   })
-  .catch((error) => {
-    Honeybadger.notify(error);
-  });
 
 initializeNav();
 
 async function loadCreatorSettings() {
-  try {
-    const [{ LogoUploadController }, { Application }] = await Promise.all([
-      import('@admin/controllers/logo_upload_controller'),
-      import('@hotwired/stimulus'),
-    ]);
+  const [{ LogoUploadController }, { Application }] = await Promise.all([
+    import('@admin/controllers/logo_upload_controller'),
+    import('@hotwired/stimulus'),
+  ]);
 
-    const application = Application.start();
-    application.register('logo-upload', LogoUploadController);
-  } catch (error) {
-    Honeybadger.notify(
-      `Error loading the creator settings controller: ${error.message}`,
-    );
-  }
+  const application = Application.start();
+  application.register('logo-upload', LogoUploadController);
 }
 
 if (document.location.pathname === '/admin/creator_settings/new') {

diff --git a/app/javascript/packs/runtimeBanner.jsx b/app/javascript/packs/runtimeBanner.jsx
@@ -21,7 +21,4 @@ waitOnBaseData()
     });
 
     loadElement();
-  })
-  .catch((error) => {
-    Honeybadger.notify(error);
   });
diff --git a/app/javascript/profileDropdown/flagButton.js b/app/javascript/profileDropdown/flagButton.js
@@ -35,10 +35,6 @@ function addFlagUserBehavior(flagButton) {
           }
         })
         .catch((e) => {
-          Honeybadger.notify(
-            isUserFlagged ? 'Unable to unflag user' : 'Unable to flag user',
-            profileUserId,
-          );
           window.alert(`Something went wrong: ${e}`);
         });
     }

diff --git a/app/javascript/sidebar-widget/SidebarWidget.jsx b/app/javascript/sidebar-widget/SidebarWidget.jsx
@@ -25,7 +25,6 @@ export const SidebarWidget = () => {
       })
       .catch((error) => {
         setSuggestedUsers([]);
-        Honeybadger.notify(error);
       });
   }, []);
 

diff --git a/app/javascript/utilities/http/csrfToken.js b/app/javascript/utilities/http/csrfToken.js
@@ -17,11 +17,6 @@ export function getCSRFToken() {
 
       if (i === MAX_RETRIES) {
         clearInterval(waitingOnCSRF);
-        Honeybadger.notify(
-          `Could not locate CSRF metatag ${JSON.stringify(
-            localStorage.current_user,
-          )}`,
-        );
         return reject(new Error('Could not locate CSRF meta tag on the page.'));
       }
     }, RETRY_INTERVAL);

diff --git a/app/javascript/utilities/sendFollowUser.js b/app/javascript/utilities/sendFollowUser.js
@@ -18,8 +18,5 @@ export function sendFollowUser(user, successCb) {
     .then((json) => {
       successCb(json.outcome);
       // json is followed or unfollowed
-    })
-    .catch((error) => {
-      Honeybadger.notify(error);
     });
 }
diff --git a/app/lib/release_phase_notifier.rb b/app/lib/release_phase_notifier.rb
@@ -9,7 +9,5 @@ def self.ping_slack
     )
 
     client.ping("Release Phase Failed: #{ENV.fetch('FAILED_COMMAND', nil)}")
-  rescue Slack::Notifier::APIError => e
-    Honeybadger.notify(e)
   end
 end
diff --git a/app/services/broadcasts/welcome_notification/generator.rb b/app/services/broadcasts/welcome_notification/generator.rb
@@ -15,8 +15,6 @@ def call
 
         notification_methods.each do |method|
           send method unless notification_enqueued # rubocop:disable Style/Send
-        rescue ActiveRecord::RecordNotFound => e
-          Honeybadger.notify(e)
         end
       end
 

diff --git a/app/services/edge_cache/bust.rb b/app/services/edge_cache/bust.rb
@@ -15,7 +15,6 @@ def call(paths)
       paths.each do |path|
         @provider_class.call(path)
       rescue StandardError => e
-        Honeybadger.notify(e)
         ForemStatsClient.increment(
           "edgecache_bust.provider_error",
           tags: ["provider_class:#{@provider_class}", "error_class:#{e.class}"],
@@ -48,7 +47,6 @@ def nginx_enabled_and_available?
       # If we can't connect to OpenResty, alert ourselves that it is
       # unavailable and return false.
       Rails.logger.error("Could not connect to OpenResty via #{ApplicationConfig['OPENRESTY_URL']}!")
-      Honeybadger.notify(e)
       ForemStatsClient.increment("edgecache_bust.service_unavailable",
                                  tags: ["path:#{ApplicationConfig['OPENRESTY_URL']}"])
     end

diff --git a/app/services/payments/customer.rb b/app/services/payments/customer.rb
@@ -71,7 +71,6 @@ def request
         ForemStatsClient.increment("stripe.errors", tags: ["error:CardError"])
         raise CardError, e.message
       rescue Stripe::StripeError => e
-        Honeybadger.notify(e)
         ForemStatsClient.increment("stripe.errors", tags: ["error:StripeError"])
         raise PaymentsError, e.message
       end

diff --git a/app/views/layouts/_logo.html.erb b/app/views/layouts/_logo.html.erb
@@ -3,7 +3,11 @@
     <img class="site-logo__img" src="<%= Settings::General.resized_logo %>" alt="<%= community_name %>">
   <% else %>
     <span class="site-logo__community-name truncate-at-2">
-      <%= community_name %>
+      <svg width="24px" height="24px" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
+  <path fill-rule="evenodd" clip-rule="evenodd"
+        d="M17.8328 7.55084L15.0044 4.72241L12.1759 7.55084L11.471 8.2558L14.2994 11.0842L15.0044 10.3793L20.6612 16.0361L23.4897 13.2077L17.8328 7.55084ZM8.99504 7.96375L11.8235 10.7922L17.4803 16.449L14.6519 19.2775L8.99504 13.6206L3.33818 19.2775L0.509766 16.449L6.16661 10.7922L8.99504 7.96375Z"
+        fill="black"/>
+</svg><%= community_name %>
     </span>
   <% end %>
 </a>
diff --git a/app/views/layouts/_user_config.html.erb b/app/views/layouts/_user_config.html.erb
@@ -35,8 +35,5 @@
       }
     }
   } catch (e) {
-    setTimeout(function () {
-      Honeybadger.notify(e);
-    }, 1000)
   }
 </script>
diff --git a/app/views/notifications/_notifications_list.html.erb b/app/views/notifications/_notifications_list.html.erb
@@ -7,8 +7,6 @@
 
   <% rescue => e %>
 
-  <% Honeybadger.notify(e, context: { notification_id: notification.id }) %>
-
   <div class="align-center p-9 py-10 color-base-80 crayons-card mb-2">
     <h2 class="fw-bold fs-l"><%= t("views.notifications.error.subtitle") %></h2>
     <p class="color-base-60 pt-2"><%= t("views.notifications.error.desc") %></p>

diff --git a/app/views/notifications/index.html.erb b/app/views/notifications/index.html.erb
@@ -33,9 +33,6 @@
 
     // TODO: [@forem/delightful]: This event doesn't appear to be firing on Honeybadger.
     // We probably want to remove it eventually if we continually don't see it being triggered.
-    if (!title) {
-      Honeybadger.notify(`Could not find parentElement.id when clicking on event target text: ${text}`);
-    }
 
     ahoy.track("Clicked Welcome Notification", { title, text, target: target.toString() });
   }

diff --git a/app/views/pages/robots.text.erb b/app/views/pages/robots.text.erb
@@ -1,18 +1,2 @@
 User-agent: *
-Disallow: /*/actions_panel*
-Disallow: /users/auth/twitter*
-Disallow: /users/auth/github*
-Disallow: /report-abuse?url=*
-Disallow: /connect/@*
-Disallow: /search?q=*
-Disallow: /search/?q=*
-Disallow: /search/feed_content?*
-Disallow: /listings*?q=*
-Disallow: /mod/*
-Disallow: /mod?*
-Disallow: /admin/*
-Disallow: /reactions?*
-Disallow: /async_info/base_data
-Disallow: /ahoy/*
-
-Sitemap: <%= URL.url("sitemap-index.xml") %>
+Disallow: /
diff --git a/app/workers/data_update_worker.rb b/app/workers/data_update_worker.rb
@@ -29,8 +29,6 @@ def run_script(script)
   rescue StandardError => e
     script.mark_as_failed!(e)
     log_status(script)
-
-    Honeybadger.notify(e, context: { script_id: script.id })
   end
 
   def log_status(script)

diff --git a/app/workers/emails/send_user_digest_worker.rb b/app/workers/emails/send_user_digest_worker.rb
@@ -11,12 +11,7 @@ def perform(user_id)
       articles = EmailDigestArticleCollector.new(user).articles_to_send
       return unless articles.any?
 
-      begin
-        DigestMailer.with(user: user, articles: articles.to_a).digest_email.deliver_now
-      rescue StandardError => e
-        Honeybadger.context({ user_id: user.id, article_ids: articles.map(&:id) })
-        Honeybadger.notify(e)
-      end
+      DigestMailer.with(user: user, articles: articles.to_a).digest_email.deliver_now
     end
   end
 end
diff --git a/app/workers/moderator/banish_user_worker.rb b/app/workers/moderator/banish_user_worker.rb
@@ -10,7 +10,6 @@ def perform(admin_id, abuser_id)
       Moderator::BanishUser.call(admin: admin, user: abuser)
     rescue StandardError => e
       ForemStatsClient.count("moderators.banishuser", 1, tags: ["action:failed", "user_id:#{abuser.id}"])
-      Honeybadger.notify(e)
     end
   end
 end
diff --git a/app/workers/moderator/sink_articles_worker.rb b/app/workers/moderator/sink_articles_worker.rb
@@ -11,7 +11,6 @@ def perform(user_id)
       Article.published.where(user: user).each(&:update_score)
     rescue StandardError => e
       ForemStatsClient.count("moderators.sink", 1, tags: ["action:failed", "user_id:#{user.id}"])
-      Honeybadger.notify(e)
     end
   end
 end