gnunet-svn
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[taler-merchant] branch master updated: rename fest: remove 'P' suffixes


From: gnunet
Subject: [taler-merchant] branch master updated: rename fest: remove 'P' suffixes from non-packable structures
Date: Mon, 09 Dec 2024 12:26:16 +0100

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

grothoff pushed a commit to branch master
in repository merchant.

The following commit(s) were added to refs/heads/master by this push:
     new a2ba9b3b rename fest: remove 'P' suffixes from non-packable structures
a2ba9b3b is described below

commit a2ba9b3b5d39a6f74e1514a675065c1d3d7e6280
Author: Christian Grothoff <christian@grothoff.org>
AuthorDate: Mon Dec 9 12:26:14 2024 +0100

    rename fest: remove 'P' suffixes from non-packable structures
---
 src/backend/taler-merchant-httpd_contract.h        |  2 +-
 .../taler-merchant-httpd_post-orders-ID-pay.c      |  8 +++---
 .../taler-merchant-httpd_private-post-orders.c     | 31 +++++++++++-----------
 src/backenddb/pg_insert_issued_token.c             |  2 +-
 src/backenddb/pg_insert_issued_token.h             |  2 +-
 src/backenddb/pg_insert_spent_token.c              |  2 +-
 src/backenddb/pg_insert_spent_token.h              |  2 +-
 src/backenddb/pg_insert_token_family_key.c         |  4 +--
 src/backenddb/pg_insert_token_family_key.h         |  4 +--
 src/backenddb/pg_lookup_spent_tokens_by_order.c    |  2 +-
 src/include/taler_merchant_service.h               | 16 +++++------
 src/include/taler_merchant_testing_lib.h           |  4 +--
 src/include/taler_merchantdb_plugin.h              | 16 +++++------
 src/testing/testing_api_cmd_pay_order.c            |  6 ++---
 14 files changed, 51 insertions(+), 50 deletions(-)

diff --git a/src/backend/taler-merchant-httpd_contract.h 
b/src/backend/taler-merchant-httpd_contract.h
index 7d29da2e..3036dc35 100644
--- a/src/backend/taler-merchant-httpd_contract.h
+++ b/src/backend/taler-merchant-httpd_contract.h
@@ -261,7 +261,7 @@ struct TALER_MerchantContractTokenFamilyKey
   /**
    * Public key.
    */
-  struct TALER_TokenIssuePublicKeyP pub;
+  struct TALER_TokenIssuePublicKey pub;
 
   /**
    * Tokens signed by this key will be valid after this time.
diff --git a/src/backend/taler-merchant-httpd_post-orders-ID-pay.c 
b/src/backend/taler-merchant-httpd_post-orders-ID-pay.c
index 9a85bb05..640731e6 100644
--- a/src/backend/taler-merchant-httpd_post-orders-ID-pay.c
+++ b/src/backend/taler-merchant-httpd_post-orders-ID-pay.c
@@ -258,7 +258,7 @@ struct TokenUseConfirmation
   /**
    * Unblinded signature on the token use public key done by the merchant.
    */
-  struct TALER_TokenIssueSignatureP unblinded_sig;
+  struct TALER_TokenIssueSignature unblinded_sig;
 
   /**
    * Hash of the token issue public key associated with this token.
@@ -297,7 +297,7 @@ struct SignedOutputToken
   /**
    * Blinded token use public keys waiting to be signed.
    */
-  struct TALER_TokenIssueBlindSignatureP sig;
+  struct TALER_BlindedTokenIssueSignature sig;
 
   /**
    * Hash of token issue public key.
@@ -2553,7 +2553,7 @@ find_valid_input_tokens (struct PayContext *pc,
 static enum GNUNET_GenericReturnValue
 sign_token_envelopes (struct PayContext *pc,
                       struct TALER_MerchantContractTokenFamilyKey *key,
-                      struct TALER_TokenIssuePrivateKeyP *priv,
+                      struct TALER_TokenIssuePrivateKey *priv,
                       bool critical,
                       unsigned int index,
                       unsigned int expected_num)
@@ -2874,7 +2874,7 @@ input_tokens_paid_check (
   const struct TALER_TokenIssuePublicKeyHashP *h_issue_pub,
   const struct TALER_TokenUsePublicKeyP *use_pub,
   const struct TALER_TokenUseSignatureP *use_sig,
-  const struct TALER_TokenIssueSignatureP *issue_sig)
+  const struct TALER_TokenIssueSignature *issue_sig)
 {
   struct PayContext *pc = cls;
 
diff --git a/src/backend/taler-merchant-httpd_private-post-orders.c 
b/src/backend/taler-merchant-httpd_private-post-orders.c
index e9abf9db..4efc1057 100644
--- a/src/backend/taler-merchant-httpd_private-post-orders.c
+++ b/src/backend/taler-merchant-httpd_private-post-orders.c
@@ -1341,7 +1341,6 @@ set_token_family (struct OrderContext *oc,
       break;
     }
   }
-
   if (NULL != family)
   {
     for (unsigned int i = 0; i<family->keys_len; i++)
@@ -1366,6 +1365,8 @@ set_token_family (struct OrderContext *oc,
                                         min_valid_after,
                                         max_valid_after,
                                         &key_details);
+  /* slug is not needed */
+  GNUNET_free (key_details.token_family.slug);
 
   if (qs <= 0)
   {
@@ -1375,10 +1376,12 @@ set_token_family (struct OrderContext *oc,
     switch (qs)
     {
     case GNUNET_DB_STATUS_HARD_ERROR:
+      GNUNET_break (0);
       http_status = MHD_HTTP_INTERNAL_SERVER_ERROR;
       ec = TALER_EC_GENERIC_DB_FETCH_FAILED;
       break;
     case GNUNET_DB_STATUS_SOFT_ERROR:
+      GNUNET_break (0);
       http_status = MHD_HTTP_INTERNAL_SERVER_ERROR;
       ec = TALER_EC_GENERIC_DB_SOFT_FAILURE;
       break;
@@ -1393,11 +1396,10 @@ set_token_family (struct OrderContext *oc,
       /* case listed to make compilers happy */
       GNUNET_assert (0);
     }
-    GNUNET_break (0);
     reply_with_error (oc,
                       http_status,
                       ec,
-                      "token_family_slug");
+                      slug);
     return GNUNET_SYSERR;
   }
 
@@ -1405,27 +1407,26 @@ set_token_family (struct OrderContext *oc,
     struct GNUNET_TIME_Timestamp now = GNUNET_TIME_timestamp_get ();
 
     /* Verify that the token family is valid right now. */
-    if (GNUNET_TIME_timestamp_cmp (key_details.token_family.valid_after, >, 
now)
-        ||
-        GNUNET_TIME_timestamp_cmp (key_details.token_family.valid_before, <=,
-                                   now)
-        )
+    if (GNUNET_TIME_timestamp_cmp (key_details.token_family.valid_after,
+                                   >,
+                                   now) ||
+        GNUNET_TIME_timestamp_cmp (key_details.token_family.valid_before,
+                                   <=,
+                                   now))
     {
       GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
-                  "Token family expired or not yet valid\n");
+                  "Token family %s expired or not yet valid\n",
+                  slug);
       reply_with_error (oc,
                         /* TODO: HTTP Status Code GONE would be more elegant,
                                  but that is already used to indicate that a 
product is out of stock. */
                         MHD_HTTP_CONFLICT,
                         
TALER_EC_MERCHANT_PRIVATE_POST_ORDERS_TOKEN_FAMILY_NOT_VALID,
-                        key_details.token_family.slug);
+                        slug);
       return GNUNET_SYSERR;
     }
   }
 
-  /* slug is not needed */
-  GNUNET_free (key_details.token_family.slug);
-
   {
     struct TALER_MerchantContractTokenFamilyKey key;
 
@@ -1500,10 +1501,10 @@ set_token_family (struct OrderContext *oc,
                                             GNUNET_CRYPTO_BSA_RSA,
                                             4096);
       {
-        struct TALER_TokenIssuePublicKeyP token_pub = {
+        struct TALER_TokenIssuePublicKey token_pub = {
           .public_key = pub,
         };
-        struct TALER_TokenIssuePrivateKeyP token_priv = {
+        struct TALER_TokenIssuePrivateKey token_priv = {
           .private_key = priv,
         };
 
diff --git a/src/backenddb/pg_insert_issued_token.c 
b/src/backenddb/pg_insert_issued_token.c
index 2c07450f..097f4f85 100644
--- a/src/backenddb/pg_insert_issued_token.c
+++ b/src/backenddb/pg_insert_issued_token.c
@@ -29,7 +29,7 @@ enum GNUNET_DB_QueryStatus
 TMH_PG_insert_issued_token (void *cls,
                             const struct TALER_PrivateContractHashP 
*h_contract_terms,
                             const struct TALER_TokenIssuePublicKeyHashP 
*h_issue_pub,
-                            const struct TALER_TokenIssueBlindSignatureP 
*blind_sig)
+                            const struct TALER_BlindedTokenIssueSignature 
*blind_sig)
 {
   struct PostgresClosure *pg = cls;
 
diff --git a/src/backenddb/pg_insert_issued_token.h 
b/src/backenddb/pg_insert_issued_token.h
index a65fc16d..7820ad27 100644
--- a/src/backenddb/pg_insert_issued_token.h
+++ b/src/backenddb/pg_insert_issued_token.h
@@ -37,6 +37,6 @@ enum GNUNET_DB_QueryStatus
 TMH_PG_insert_issued_token (void *cls,
                           const struct TALER_PrivateContractHashP 
*h_contract_terms,
                           const struct TALER_TokenIssuePublicKeyHashP 
*h_issue_pub,
-                          const struct TALER_TokenIssueBlindSignatureP 
*blind_sig);
+                          const struct TALER_BlindedTokenIssueSignature 
*blind_sig);
 
 #endif
diff --git a/src/backenddb/pg_insert_spent_token.c 
b/src/backenddb/pg_insert_spent_token.c
index bb3ffcdc..0cf7efb9 100644
--- a/src/backenddb/pg_insert_spent_token.c
+++ b/src/backenddb/pg_insert_spent_token.c
@@ -33,7 +33,7 @@ TMH_PG_insert_spent_token (
   const struct TALER_TokenIssuePublicKeyHashP *h_issue_pub,
   const struct TALER_TokenUsePublicKeyP *use_pub,
   const struct TALER_TokenUseSignatureP *use_sig,
-  const struct TALER_TokenIssueSignatureP *issue_sig)
+  const struct TALER_TokenIssueSignature *issue_sig)
 {
   struct PostgresClosure *pg = cls;
   struct GNUNET_PQ_QueryParam params[] = {
diff --git a/src/backenddb/pg_insert_spent_token.h 
b/src/backenddb/pg_insert_spent_token.h
index c618213f..aa7d44dc 100644
--- a/src/backenddb/pg_insert_spent_token.h
+++ b/src/backenddb/pg_insert_spent_token.h
@@ -42,6 +42,6 @@ TMH_PG_insert_spent_token (
   const struct TALER_TokenIssuePublicKeyHashP *h_issue_pub,
   const struct TALER_TokenUsePublicKeyP *use_pub,
   const struct TALER_TokenUseSignatureP *use_sig,
-  const struct TALER_TokenIssueSignatureP *issue_sig);
+  const struct TALER_TokenIssueSignature *issue_sig);
 
 #endif
diff --git a/src/backenddb/pg_insert_token_family_key.c 
b/src/backenddb/pg_insert_token_family_key.c
index f862a120..1f441e49 100644
--- a/src/backenddb/pg_insert_token_family_key.c
+++ b/src/backenddb/pg_insert_token_family_key.c
@@ -33,8 +33,8 @@ TMH_PG_insert_token_family_key (
   void *cls,
   const char *merchant_id,
   const char *token_family_slug,
-  const struct TALER_TokenIssuePublicKeyP *pub,
-  const struct TALER_TokenIssuePrivateKeyP *priv,
+  const struct TALER_TokenIssuePublicKey *pub,
+  const struct TALER_TokenIssuePrivateKey *priv,
   const struct GNUNET_TIME_Timestamp valid_after,
   const struct GNUNET_TIME_Timestamp valid_before)
 {
diff --git a/src/backenddb/pg_insert_token_family_key.h 
b/src/backenddb/pg_insert_token_family_key.h
index cedbf81b..c8c85351 100644
--- a/src/backenddb/pg_insert_token_family_key.h
+++ b/src/backenddb/pg_insert_token_family_key.h
@@ -43,8 +43,8 @@ TMH_PG_insert_token_family_key (
   void *cls,
   const char *merchant_id,
   const char *token_family_slug,
-  const struct TALER_TokenIssuePublicKeyP *pub,
-  const struct TALER_TokenIssuePrivateKeyP *priv,
+  const struct TALER_TokenIssuePublicKey *pub,
+  const struct TALER_TokenIssuePrivateKey *priv,
   const struct GNUNET_TIME_Timestamp valid_after,
   const struct GNUNET_TIME_Timestamp valid_before);
 
diff --git a/src/backenddb/pg_lookup_spent_tokens_by_order.c 
b/src/backenddb/pg_lookup_spent_tokens_by_order.c
index 5c27e072..838b232e 100644
--- a/src/backenddb/pg_lookup_spent_tokens_by_order.c
+++ b/src/backenddb/pg_lookup_spent_tokens_by_order.c
@@ -76,7 +76,7 @@ lookup_spent_tokens_by_order_cb (void *cls,
     struct TALER_TokenIssuePublicKeyHashP h_issue_pub;
     struct TALER_TokenUsePublicKeyP use_pub;
     struct TALER_TokenUseSignatureP use_sig;
-    struct TALER_TokenIssueSignatureP issue_sig;
+    struct TALER_TokenIssueSignature issue_sig;
     struct GNUNET_PQ_ResultSpec rs[] = {
       GNUNET_PQ_result_spec_uint64 ("spent_token_serial",
                                     &spent_token_serial),
diff --git a/src/include/taler_merchant_service.h 
b/src/include/taler_merchant_service.h
index 78edc62f..a97c1eb7 100644
--- a/src/include/taler_merchant_service.h
+++ b/src/include/taler_merchant_service.h
@@ -3167,17 +3167,17 @@ struct TALER_MERCHANT_PrivateTokenDetails
   /**
    * Token issue public key.
    */
-  struct TALER_TokenIssuePublicKeyP issue_pub;
+  struct TALER_TokenIssuePublicKey issue_pub;
 
   /**
    * Unblinded token issue signature made by the merchant.
    */
-  struct TALER_TokenIssueSignatureP issue_sig;
+  struct TALER_TokenIssueSignature issue_sig;
 
   /**
    * Blinded token issue signature made by the merchant.
    */
-  struct TALER_TokenIssueBlindSignatureP blinded_sig;
+  struct TALER_BlindedTokenIssueSignature blinded_sig;
 
 };
 
@@ -3343,12 +3343,12 @@ struct TALER_MERCHANT_UsedToken
   /**
    * Unblinded signature made by the token issue public key of the merchant.
    */
-  struct TALER_TokenIssueSignatureP ub_sig;
+  struct TALER_TokenIssueSignature ub_sig;
 
   /**
    * Token issue public key associated with this token.
    */
-  struct TALER_TokenIssuePublicKeyP issue_pub;
+  struct TALER_TokenIssuePublicKey issue_pub;
 
 };
 
@@ -3369,7 +3369,7 @@ struct TALER_MERCHANT_OutputToken
   /**
    * Blinded issue signature made by the merchant.
    */
-  struct TALER_TokenIssueBlindSignatureP blinded_sig;
+  struct TALER_BlindedTokenIssueSignature blinded_sig;
 
 };
 
@@ -3475,12 +3475,12 @@ struct TALER_MERCHANT_UseToken
   /**
    * Unblinded signature made by the token issue public key of the merchant.
    */
-  struct TALER_TokenIssueSignatureP ub_sig;
+  struct TALER_TokenIssueSignature ub_sig;
 
   /**
    * Token issue public key associated with this token.
    */
-  struct TALER_TokenIssuePublicKeyP issue_pub;
+  struct TALER_TokenIssuePublicKey issue_pub;
 
 };
 
diff --git a/src/include/taler_merchant_testing_lib.h 
b/src/include/taler_merchant_testing_lib.h
index 66da80db..d3bf0d1c 100644
--- a/src/include/taler_merchant_testing_lib.h
+++ b/src/include/taler_merchant_testing_lib.h
@@ -1867,8 +1867,8 @@ TALER_TESTING_cmd_checkserver2 (const char *label,
         op (http_body_size, const size_t)   \
         op (planchet_secrets, const struct TALER_PlanchetMasterSecretP)   \
         op (token_priv, const struct TALER_TokenUsePrivateKeyP)   \
-        op (token_issue_sig, const struct TALER_TokenIssueSignatureP)   \
-        op (token_issue_pub, const struct TALER_TokenIssuePublicKeyP)
+        op (token_issue_sig, const struct TALER_TokenIssueSignature)   \
+        op (token_issue_pub, const struct TALER_TokenIssuePublicKey)
 
 
 TALER_MERCHANT_TESTING_SIMPLE_TRAITS (TALER_TESTING_MAKE_DECL_SIMPLE_TRAIT)
diff --git a/src/include/taler_merchantdb_plugin.h 
b/src/include/taler_merchantdb_plugin.h
index 3c3d6cb3..60590047 100644
--- a/src/include/taler_merchantdb_plugin.h
+++ b/src/include/taler_merchantdb_plugin.h
@@ -1264,12 +1264,12 @@ struct TALER_MERCHANTDB_TokenFamilyKeyDetails
   /**
    * Token family public key.
    */
-  struct TALER_TokenIssuePublicKeyP pub;
+  struct TALER_TokenIssuePublicKey pub;
 
   /**
    * Token family private key.
    */
-  struct TALER_TokenIssuePrivateKeyP priv;
+  struct TALER_TokenIssuePrivateKey priv;
 
   /**
    * Details about the token family this key belongs to.
@@ -1295,7 +1295,7 @@ struct TALER_MERCHANTDB_SpentTokenDetails
   /**
    * Blind signature for the spent token to prove validity of it.
   */
-  struct TALER_TokenIssueBlindSignatureP blind_sig;
+  struct TALER_BlindedTokenIssueSignature blind_sig;
 };
 
 
@@ -1318,7 +1318,7 @@ typedef void
   const struct TALER_TokenIssuePublicKeyHashP *h_issue_pub,
   const struct TALER_TokenUsePublicKeyP *use_pub,
   const struct TALER_TokenUseSignatureP *use_sig,
-  const struct TALER_TokenIssueSignatureP *issue_sig);
+  const struct TALER_TokenIssueSignature *issue_sig);
 
 /**
  * Handle to interact with the database.
@@ -2722,7 +2722,7 @@ struct TALER_MERCHANTDB_Plugin
     const struct TALER_TokenIssuePublicKeyHashP *h_issue_pub,
     const struct TALER_TokenUsePublicKeyP *use_pub,
     const struct TALER_TokenUseSignatureP *use_sig,
-    const struct TALER_TokenIssueSignatureP *issue_sig);
+    const struct TALER_TokenIssueSignature *issue_sig);
 
 
   /**
@@ -2739,7 +2739,7 @@ struct TALER_MERCHANTDB_Plugin
     void *cls,
     const struct TALER_PrivateContractHashP *h_contract_terms,
     const struct TALER_TokenIssuePublicKeyHashP *h_issue_pub,
-    const struct TALER_TokenIssueBlindSignatureP *blind_sig);
+    const struct TALER_BlindedTokenIssueSignature *blind_sig);
 
 
   /**
@@ -3772,8 +3772,8 @@ struct TALER_MERCHANTDB_Plugin
     void *cls,
     const char *merchant_id,
     const char *token_family_slug,
-    const struct TALER_TokenIssuePublicKeyP *pub,
-    const struct TALER_TokenIssuePrivateKeyP *priv,
+    const struct TALER_TokenIssuePublicKey *pub,
+    const struct TALER_TokenIssuePrivateKey *priv,
     struct GNUNET_TIME_Timestamp valid_after,
     struct GNUNET_TIME_Timestamp valid_before);
 
diff --git a/src/testing/testing_api_cmd_pay_order.c 
b/src/testing/testing_api_cmd_pay_order.c
index e9a893e3..dc27b842 100644
--- a/src/testing/testing_api_cmd_pay_order.c
+++ b/src/testing/testing_api_cmd_pay_order.c
@@ -152,7 +152,7 @@ static enum GNUNET_GenericReturnValue
 find_token_public_key (const json_t *token_families,
                        const char *slug,
                        struct GNUNET_TIME_Timestamp valid_after,
-                       struct TALER_TokenIssuePublicKeyP *pub)
+                       struct TALER_TokenIssuePublicKey *pub)
 {
   const json_t *tf = json_object_get (token_families, slug);
   const json_t *keys;
@@ -443,8 +443,8 @@ build_tokens (struct TALER_MERCHANT_UseToken **tokens,
 
     {
       const struct TALER_TokenUsePrivateKeyP *token_priv;
-      const struct TALER_TokenIssueSignatureP *issue_sig;
-      const struct TALER_TokenIssuePublicKeyP *issue_pub;
+      const struct TALER_TokenIssueSignature *issue_sig;
+      const struct TALER_TokenIssuePublicKey *issue_pub;
 
       GNUNET_assert (GNUNET_OK ==
                      TALER_TESTING_get_trait_token_priv (pay_cmd,

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]