diff --git a/lib/plantid_discord_bot/cogs/plantnet.ex b/lib/plantid_discord_bot/cogs/plantnet.ex index 0d84ab4..1e992a1 100644 --- a/lib/plantid_discord_bot/cogs/plantnet.ex +++ b/lib/plantid_discord_bot/cogs/plantnet.ex @@ -51,6 +51,7 @@ defmodule PlantIdDiscordBot.Cog.PlantNet do }) end + # TODO remove Logger.debug(saved_images) # TODO use actual image data @@ -74,10 +75,15 @@ defmodule PlantIdDiscordBot.Cog.PlantNet do }) {:ok, %HTTPoison.Response{status_code: 400}} -> - Logger.error("Malformed request sent to the PlantNet API") + Logger.error(""" + Malformed request sent to the PlantNet Api from PlantIdDiscordBot.Cog.PlantNet.do_identification + Query URI: #{query_uri} + Status Code: 400 + Response Body: #{String.slice(body, 0..500)}... (truncated) + """) Api.create_followup_message(interaction.application_id, interaction.token, %{ - content: "Bad Request" + content: "Bad Request. This error has been logged." }) {:ok, %HTTPoison.Response{status_code: 404}} -> diff --git a/lib/plantid_discord_bot/rate_limiter.ex b/lib/plantid_discord_bot/rate_limiter.ex index 8c38553..ca2b996 100644 --- a/lib/plantid_discord_bot/rate_limiter.ex +++ b/lib/plantid_discord_bot/rate_limiter.ex @@ -1,5 +1,6 @@ defmodule PlantIdDiscordBot.RateLimiter do use GenServer + require Logger @request_limit Application.compile_env(:plantid_discord_bot, :guild_request_limit_per_day) @@ -48,8 +49,13 @@ defmodule PlantIdDiscordBot.RateLimiter do """ def increase_counter(guild_id) do case get(guild_id) do - {:ok, _} -> :ets.update_counter(__MODULE__, guild_id, 1) - {:error, _} -> put(guild_id, 1) + {:ok, _} -> + :ets.update_counter(__MODULE__, guild_id, 1) + Logger.debug("Increased counter for guild #{guild_id}") + + {:error, _} -> + put(guild_id, 1) + Logger.debug("Set counter for guild #{guild_id} to 1") end end @@ -57,5 +63,8 @@ defmodule PlantIdDiscordBot.RateLimiter do Reset all guilds counters to 0. """ @spec reset_counters() :: :ok - def reset_counters(), do: :ets.delete_all_objects(__MODULE__) + def reset_counters() do + :ets.delete_all_objects(__MODULE__) + Logger.debug("Reset all guilds counters") + end end