From 7995678e776b719145aeaf9e4409b6dd694c253e Mon Sep 17 00:00:00 2001 From: Christian Grothoff Date: Fri, 17 Jan 2020 03:42:31 +0100 Subject: fix error code use --- src/exchange/taler-exchange-httpd_keystate.c | 11 ++++------- src/exchange/taler-exchange-httpd_refresh_reveal.c | 3 +-- 2 files changed, 5 insertions(+), 9 deletions(-) (limited to 'src/exchange') diff --git a/src/exchange/taler-exchange-httpd_keystate.c b/src/exchange/taler-exchange-httpd_keystate.c index 4115fe66e..31cb13684 100644 --- a/src/exchange/taler-exchange-httpd_keystate.c +++ b/src/exchange/taler-exchange-httpd_keystate.c @@ -2285,11 +2285,6 @@ TEH_KS_free () /** * Sign the message in @a purpose with the exchange's signing key. * - * FIXME: - * - Change API to return status code and do not assert on TEH_KS_acquire() - * failures, instead allow caller to handle it (i.e. by returning - * #TALER_EC_EXCHANGE_BAD_CONFIGURATION to application). - * * @param purpose the message to sign * @param[out] pub set to the current public signing key of the exchange * @param[out] sig signature over purpose using current signing key @@ -2451,9 +2446,11 @@ TEH_KS_handler_keys (struct TEH_RequestHandler *rh, } if (NULL == krd) { - /* FIXME: should return 500 response instead... */ GNUNET_break (0); - return MHD_NO; + return TALER_MHD_reply_with_error (connection, + MHD_HTTP_INTERNAL_SERVER_ERROR, + TALER_EC_KEYS_MISSING, + "no key response found"); } ret = MHD_queue_response (connection, rh->response_code, diff --git a/src/exchange/taler-exchange-httpd_refresh_reveal.c b/src/exchange/taler-exchange-httpd_refresh_reveal.c index fb2602c61..9a8d929fb 100644 --- a/src/exchange/taler-exchange-httpd_refresh_reveal.c +++ b/src/exchange/taler-exchange-httpd_refresh_reveal.c @@ -608,10 +608,9 @@ handle_refresh_reveal_json (struct MHD_Connection *connection, if (NULL == key_state) { TALER_LOG_ERROR ("Lacking keys to operate\n"); - /* FIXME: use correct EC code! */ return TALER_MHD_reply_with_error (connection, MHD_HTTP_INTERNAL_SERVER_ERROR, - TALER_EC_REFRESH_REVEAL_SIGNING_ERROR, + TALER_EC_REFRESH_REVEAL_KEYS_MISSING, "exchange lacks keys"); } -- cgit v1.2.3