[taler-exchange] 89/151: -improve logging

[gnunet]

This is an automated email from the git hooks/post-receive script.

grothoff pushed a commit to branch master
in repository exchange.

commit 4545732a229676d1359c380f16dc8424d1e9d673
Author: Christian Grothoff <christian@grothoff.org>
AuthorDate: Wed Jul 17 09:11:42 2024 +0200

    -improve logging
---
 src/exchange/taler-exchange-httpd_batch-withdraw.c | 6 +++++-
 src/exchange/taler-exchange-httpd_withdraw.c       | 4 ++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/exchange/taler-exchange-httpd_batch-withdraw.c 
b/src/exchange/taler-exchange-httpd_batch-withdraw.c
index d9f5489d8..1b473f2a3 100644
--- a/src/exchange/taler-exchange-httpd_batch-withdraw.c
+++ b/src/exchange/taler-exchange-httpd_batch-withdraw.c
@@ -511,6 +511,7 @@ parse_planchets (const struct TEH_RequestContext *rc,
     if (! check_request_idempotent (wc,
                                     &mret))
     {
+      GNUNET_break (0);
       return TALER_MHD_reply_with_error (rc->connection,
                                          MHD_HTTP_INTERNAL_SERVER_ERROR,
                                          
TALER_EC_EXCHANGE_GENERIC_KEYS_MISSING,
@@ -528,7 +529,6 @@ parse_planchets (const struct TEH_RequestContext *rc,
       &pc->collectable.denom_pub_hash,
       NULL,
       NULL);
-
     if (NULL == dk)
     {
       MHD_RESULT mret;
@@ -536,6 +536,7 @@ parse_planchets (const struct TEH_RequestContext *rc,
       if (! check_request_idempotent (wc,
                                       &mret))
       {
+        GNUNET_break_op (0);
         return TEH_RESPONSE_reply_unknown_denom_pub_hash (
           rc->connection,
           &pc->collectable.denom_pub_hash);
@@ -550,6 +551,7 @@ parse_planchets (const struct TEH_RequestContext *rc,
       if (! check_request_idempotent (wc,
                                       &mret))
       {
+        GNUNET_break_op (0);
         return TEH_RESPONSE_reply_expired_denom_pub_hash (
           rc->connection,
           &pc->collectable.denom_pub_hash,
@@ -562,6 +564,7 @@ parse_planchets (const struct TEH_RequestContext *rc,
     {
       /* This denomination is not yet valid, no need to check
          for idempotency! */
+      GNUNET_break_op (0);
       return TEH_RESPONSE_reply_expired_denom_pub_hash (
         rc->connection,
         &pc->collectable.denom_pub_hash,
@@ -576,6 +579,7 @@ parse_planchets (const struct TEH_RequestContext *rc,
       if (! check_request_idempotent (wc,
                                       &mret))
       {
+        GNUNET_break_op (0);
         return TEH_RESPONSE_reply_expired_denom_pub_hash (
           rc->connection,
           &pc->collectable.denom_pub_hash,
diff --git a/src/exchange/taler-exchange-httpd_withdraw.c 
b/src/exchange/taler-exchange-httpd_withdraw.c
index 1516d8f54..b0934ab7a 100644
--- a/src/exchange/taler-exchange-httpd_withdraw.c
+++ b/src/exchange/taler-exchange-httpd_withdraw.c
@@ -17,8 +17,8 @@
   see <http://www.gnu.org/licenses/>
 */
 /**
- * @file taler-exchange-httpd_batch-withdraw.c
- * @brief Handle /reserves/$RESERVE_PUB/batch-withdraw requests
+ * @file taler-exchange-httpd_withdraw.c
+ * @brief Common logic for withdraw operations
  * @author Florian Dold
  * @author Benedikt Mueller
  * @author Christian Grothoff

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.