diff options
| author | Christian Grothoff <christian@grothoff.org> | 2020-01-17 03:42:31 +0100 |
|---|---|---|
| committer | Christian Grothoff <christian@grothoff.org> | 2020-01-17 03:42:36 +0100 |
| commit | 7995678e776b719145aeaf9e4409b6dd694c253e (patch) | |
| tree | f2fe1d83dd130f63c9b1379fa02477857bd32ecd /src/exchange/taler-exchange-httpd_keystate.c | |
| parent | dcc80fd35dccf7c4686d267d37c95c38eb5fd36f (diff) | |
| download | exchange-7995678e776b719145aeaf9e4409b6dd694c253e.tar.gz exchange-7995678e776b719145aeaf9e4409b6dd694c253e.tar.bz2 exchange-7995678e776b719145aeaf9e4409b6dd694c253e.zip | |
fix error code use
Diffstat (limited to 'src/exchange/taler-exchange-httpd_keystate.c')
| -rw-r--r-- | src/exchange/taler-exchange-httpd_keystate.c | 11 |
1 files changed, 4 insertions, 7 deletions
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, |