exchange

commit 022fba51d569826a376f517e5ee3302222e7470c
parent ccdbc2a4526c5f8750cdce2180b4d5f481d264b4
Author: Christian Grothoff <christian@grothoff.org>
Date:   Mon, 27 Jul 2020 14:14:07 +0200

better fix

Diffstat:
M
src/bank-lib/taler-bank-transfer.c
 | 
4
++++
M
src/include/taler_util.h
 | 
9
+++++++++
M
src/util/payto.c
 | 
39
+++++++++++++++++++++++++++++++++++++++
M
src/util/test_payto.c
 | 
11
+++++++++++

4 files changed, 63 insertions(+), 0 deletions(-)
diff --git a/src/bank-lib/taler-bank-transfer.c b/src/bank-lib/taler-bank-transfer.c
@@ -409,6 +409,10 @@ execute_wire_transfer ()
     GNUNET_SCHEDULER_shutdown ();
     return;
   }
+
+  // See if subject was given as a payto-parameter.
+  if (NULL == subject)
+    subject = TALER_payto_get_subject (credit_account);
   if (NULL != subject)
   {
     if (GNUNET_OK !=
diff --git a/src/include/taler_util.h b/src/include/taler_util.h
@@ -297,6 +297,15 @@ TALER_payto_get_method (const char *payto_uri);
 char *
 TALER_xtalerbank_account_from_payto (const char *payto);
 
+/**
+ * Extract the subject value from the URI parameters.
+ *
+ * @param payto_uri the URL to parse
+ * @return NULL if the subject parameter is not found.
+ *         The caller should free the returned value.
+ */
+char *
+TALER_payto_get_subject (const char *payto_uri);
 
 /**
  * Possible values for a binary filter.
diff --git a/src/util/payto.c b/src/util/payto.c
@@ -29,6 +29,45 @@
 
 
 /**
+ * Extract the subject value from the URI parameters.
+ *
+ * @param payto_uri the URL to parse
+ * @return NULL if the subject parameter is not found.
+ *         The caller should free the returned value.
+ */
+char *
+TALER_payto_get_subject (const char *payto_uri)
+{
+  const char *key;
+  const char *value_start;
+  const char *value_end;
+
+  key = strchr (payto_uri,
+                (unsigned char) '?');
+  if (NULL == key)
+    return NULL;
+
+  do {
+    if (0 == strncasecmp (++key,
+			  "subject",
+			  strlen ("subject")))
+    {
+      value_start = strchr (key,
+	                    (unsigned char) '=');
+      if (NULL == value_start)
+        return NULL;
+      value_end = strchrnul (value_start,
+	                     (unsigned char) '&');
+
+      return GNUNET_strndup (value_start + 1,
+	                     value_end - value_start - 1);
+    }
+  } while ( (key = strchr (key,
+			 (unsigned char) '&')) );
+  return NULL;
+}
+
+/**
  * Obtain the payment method from a @a payto_uri. The
  * format of a payto URI is 'payto://$METHOD/$SOMETHING'.
  * We return $METHOD.
diff --git a/src/util/test_payto.c b/src/util/test_payto.c
@@ -22,6 +22,8 @@
 #include "taler_util.h"
 
 #define CHECK(a,b) do { \
+    GNUNET_assert (a != NULL); \
+    GNUNET_assert (b != NULL); \
     if (0 != strcmp (a,b)) {   \
       GNUNET_break (0); \
       fprintf (stderr, "Got %s, wanted %s\n", b, a); \
@@ -52,6 +54,15 @@ main (int argc,
     "payto://x-taler-bank/localhost:1080/alice?subject=hello&amount=EUR:1");
   CHECK ("alice",
          r);
+
+  r = TALER_payto_get_subject (
+    "payto://x-taler-bank/localhost:1080/alice?subject=hello&amount=EUR:1");
+  CHECK ("hello",
+	 r);
+
+  r = TALER_payto_get_subject (
+    "payto://x-taler-bank/localhost:1080/alice");
+  GNUNET_assert (r == NULL);
   return 0;
 }