From 0e90ee527cbff779d99a51e4b9773b5a29f13b98 Mon Sep 17 00:00:00 2001 From: Shadowfacts Date: Wed, 14 Sep 2022 18:12:12 -0400 Subject: [PATCH] Remove Sentry breadcrumbs from network requests There ends up being a multi-hour gap between the breadcrumbs listed and the error itself, so they're useless. --- lib/frenzy/network.ex | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/lib/frenzy/network.ex b/lib/frenzy/network.ex index afcd214..3df7952 100644 --- a/lib/frenzy/network.ex +++ b/lib/frenzy/network.ex @@ -24,19 +24,11 @@ defmodule Frenzy.Network do @spec http_get(String.t()) :: Tesla.Env.result() def http_get(url) do - if Frenzy.sentry_enabled?() do - Sentry.Context.add_breadcrumb(%{category: "http_get", message: url}) - end - HTTP.get(url) end @spec http_post(String.t(), Tesla.Env.body(), [Tesla.option()]) :: Tesla.Env.result() def http_post(url, body, options \\ []) do - if Frenzy.sentry_enabled?() do - Sentry.Context.add_breadcrumb(%{category: "http_get", message: url}) - end - HTTP.post(url, body, options) end @@ -91,10 +83,6 @@ defmodule Frenzy.Network do @spec gemini_request(String.t() | URI.t()) :: {:ok, Gemini.Response.t()} | {:error, term()} def gemini_request(uri) do - if Frenzy.sentry_enabled?() do - Sentry.Context.add_breadcrumb(%{category: "gemini_request", message: uri}) - end - case Gemini.request(uri) do {:ok, %Gemini.Response{status: code} = response} when code in @gemini_success_codes -> {:ok, response}