[taler-exchange] branch master updated: fix #6969

[gnunet]

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

grothoff pushed a commit to branch master
in repository exchange.

The following commit(s) were added to refs/heads/master by this push:
     new 5fc2e72a fix #6969
5fc2e72a is described below

commit 5fc2e72a2a42434495237e7551af8ec054718171
Author: Christian Grothoff <christian@grothoff.org>
AuthorDate: Tue Aug 3 14:41:42 2021 +0200

    fix #6969
---
 src/bank-lib/taler-exchange-wire-gateway-client.c | 21 ++++++++++++++++-----
 src/util/url.c                                    |  3 ++-
 2 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/src/bank-lib/taler-exchange-wire-gateway-client.c 
b/src/bank-lib/taler-exchange-wire-gateway-client.c
index 14af7fb2..84277808 100644
--- a/src/bank-lib/taler-exchange-wire-gateway-client.c
+++ b/src/bank-lib/taler-exchange-wire-gateway-client.c
@@ -606,7 +606,7 @@ run (void *cls,
          (NULL != auth.details.basic.password) )
     {
       fprintf (stderr,
-               "Conflicting authentication options provided. Please only use 
one method.\n");
+               "Error: Conflicting authentication options provided. Please 
only use one method.\n");
       GNUNET_SCHEDULER_shutdown ();
       return;
     }
@@ -616,7 +616,7 @@ run (void *cls,
                                    &auth))
     {
       fprintf (stderr,
-               "Authentication information not found in configuration section 
`%s'\n",
+               "Error: Authentication information not found in configuration 
section `%s'\n",
                account_section);
       GNUNET_SCHEDULER_shutdown ();
       return;
@@ -633,16 +633,27 @@ run (void *cls,
     else if (NULL == auth.wire_gateway_url)
     {
       fprintf (stderr,
-               "No account specified (use -b or -s options).\n");
+               "Error: No account specified (use -b or -s options).\n");
       GNUNET_SCHEDULER_shutdown ();
       return;
     }
   }
+  if ( (NULL == auth.wire_gateway_url) ||
+       (0 == strlen (auth.wire_gateway_url)) ||
+       (0 != strcasecmp ("http",
+                         auth.wire_gateway_url)) )
+  {
+    fprintf (stderr,
+             "Error: Invalid wire gateway URL `%s' configured.\n",
+             auth.wire_gateway_url);
+    GNUNET_SCHEDULER_shutdown ();
+    return;
+  }
   if ( (GNUNET_YES == incoming_history) &&
        (GNUNET_YES == outgoing_history) )
   {
     fprintf (stderr,
-             "Please specify only -i or -o, but not both.\n");
+             "Error: Please specify only -i or -o, but not both.\n");
     GNUNET_SCHEDULER_shutdown ();
     return;
   }
@@ -675,7 +686,7 @@ run (void *cls,
 
 
 /**
- * The main function of the taler-bank-transfer tool
+ * The main function of the taler-exchange-wire-gateway-client
  *
  * @param argc number of arguments from the command line
  * @param argv command line arguments
diff --git a/src/util/url.c b/src/util/url.c
index 3fbc4a04..cae36299 100644
--- a/src/util/url.c
+++ b/src/util/url.c
@@ -240,7 +240,8 @@ TALER_url_join (const char *base_url,
   if (0 == strlen (base_url))
   {
     /* base URL can't be empty */
-    GNUNET_break (0);
+    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                "Empty base URL specified\n");
     return NULL;
   }
   if ('/' != base_url[strlen (base_url) - 1])

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