From a94048761129d3f1cd262f9179b7c06d875c9bcf Mon Sep 17 00:00:00 2001
From: Greyson Parrelli <greyson@signal.org>
Date: Thu, 2 Sep 2021 14:09:48 -0400
Subject: [PATCH] Improve logging around rate-limiting.

---
 .../main/java/org/thoughtcrime/securesms/jobs/PushSendJob.java  | 2 ++
 .../securesms/ratelimit/RecaptchaProofBottomSheetFragment.java  | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/PushSendJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/PushSendJob.java
index e6e3c8b5e..01cac5a26 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/jobs/PushSendJob.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/PushSendJob.java
@@ -467,6 +467,8 @@ public abstract class PushSendJob extends SendJob {
   protected void handleProofRequiredException(@NonNull ProofRequiredException proofRequired, @Nullable Recipient recipient, long threadId, long messageId, boolean isMms)
       throws ProofRequiredException, RetryLaterException
   {
+    Log.w(TAG, "[Proof Required] Options: " + proofRequired.getOptions());
+
     try {
       if (proofRequired.getOptions().contains(ProofRequiredException.Option.PUSH_CHALLENGE)) {
         ApplicationDependencies.getSignalServiceAccountManager().requestRateLimitPushChallenge();
diff --git a/app/src/main/java/org/thoughtcrime/securesms/ratelimit/RecaptchaProofBottomSheetFragment.java b/app/src/main/java/org/thoughtcrime/securesms/ratelimit/RecaptchaProofBottomSheetFragment.java
index 13ede1810..5d7721d00 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/ratelimit/RecaptchaProofBottomSheetFragment.java
+++ b/app/src/main/java/org/thoughtcrime/securesms/ratelimit/RecaptchaProofBottomSheetFragment.java
@@ -48,6 +48,8 @@ public final class RecaptchaProofBottomSheetFragment extends BottomSheetDialogFr
 
   @Override
   public void show(@NonNull FragmentManager manager, @Nullable String tag) {
+    Log.i(TAG, "Showing reCAPTCHA proof bottom sheet.");
+
     if (manager.findFragmentByTag(tag) == null) {
       BottomSheetUtil.show(manager, tag, this);
     } else {