From f5f7374427fffcf7d6893efab87518a2a0c54478 Mon Sep 17 00:00:00 2001
From: Jean-Paul Calderone <exarkun@twistedmatrix.com>
Date: Fri, 27 Sep 2019 10:09:56 -0400
Subject: [PATCH] more logging in the ristretto redeemer

---
 src/_zkapauthorizer/controller.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/_zkapauthorizer/controller.py b/src/_zkapauthorizer/controller.py
index 3303892..decee7a 100644
--- a/src/_zkapauthorizer/controller.py
+++ b/src/_zkapauthorizer/controller.py
@@ -181,6 +181,8 @@ class DummyRedeemer(object):
 @implementer(IRedeemer)
 @attr.s
 class RistrettoRedeemer(object):
+    _log = Logger()
+
     _treq = attr.ib()
     _api_root = attr.ib(validator=attr.validators.instance_of(URL))
 
@@ -217,7 +219,12 @@ class RistrettoRedeemer(object):
                 ),
             }),
         )
-        result = yield json_content(response)
+        try:
+            result = yield json_content(response)
+        except ValueError:
+            self._log.failure("Parsing redeem response failed", response=response)
+            raise TransientRedemptionError()
+
         marshaled_signed_tokens = result[u"signatures"]
         marshaled_proof = result[u"proof"]
         marshaled_public_key = result[u"public-key"]
-- 
GitLab